-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Rename Task Success Evaluator to Task Completion Evaluator #43190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename Task Success Evaluator to Task Completion Evaluator #43190
Conversation
Thank you for your contribution @salma-elshafey! We will review the pull request and get back to you soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request renames the "Task Success Evaluator" to "Task Completion Evaluator" across the Azure AI Evaluation SDK. The rename affects class names, file paths, imports, and documentation to consistently use the new terminology.
Key changes:
- Renamed TaskSuccessEvaluator class to TaskCompletionEvaluator
- Updated file and directory structure to use "task_completion" naming
- Modified sample code and documentation to reflect the new class name
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
test_aoai_score_model_grader.py | Added skip markers to test methods |
evaluation_samples_evaluate_fdp.py | Added sample code for TaskCompletionEvaluator |
evaluation_samples_evaluate.py | Fixed import and added TaskCompletionEvaluator sample |
agent_evaluation.ipynb | Removed unused import and trailing commas |
cspell.json | Added new words and updated file paths for task_completion |
_exceptions.py | Added TASK_COMPLETION_EVALUATOR error target |
task_completion.prompty | New prompty file for task completion evaluation |
_task_completion.py | New evaluator implementation for task completion |
init.py | New module init file exporting TaskCompletionEvaluator |
_eval_mapping.py | Added TaskCompletionEvaluator to evaluation mapping |
...ion/azure-ai-evaluation/azure/ai/evaluation/_evaluators/_task_completion/_task_completion.py
Outdated
Show resolved
Hide resolved
...ion/azure-ai-evaluation/azure/ai/evaluation/_evaluators/_task_completion/_task_completion.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new API spec, a link to the pull request containing these API spec changes should be included above.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines