Skip to content

Evaluation: Remove parallel from composite evaluators #38168

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.

Sign up for GitHub

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

Merged
ninghu merged 12 commits into from
Oct 30, 2024
Merged

Evaluation: Remove parallel from composite evaluators #38168

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
094b440
Remove `parallel` from composite evaluators
ninghu Oct 29, 2024
fd8881c
Merge branch 'main' into ninhu/parallel
ninghu Oct 29, 2024
7949973
update recording
ninghu Oct 29, 2024
3c17258
update
ninghu Oct 29, 2024
04e4afc
output_dir check
ninghu Oct 30, 2024
07d49cf
fix the test recording
ninghu Oct 30, 2024
9c09c87
fix the failed unit-test
ninghu Oct 30, 2024
d7382d6
update changelog
ninghu Oct 30, 2024
796ad1e
update
ninghu Oct 30, 2024
7862ad5
fix black issue
ninghu Oct 30, 2024
4ee1fb9
revert output_path related change
ninghu Oct 30, 2024
55eba1e
Update sdk/evaluation/azure-ai-evaluation/CHANGELOG.md
ninghu Oct 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Update sdk/evaluation/azure-ai-evaluation/CHANGELOG.md 
Co-authored-by: Neehar Duvvuri <[email protected]>
  • Loading branch information
@ninghu @needuv
ninghu and needuv authored Oct 30, 2024
commit 55eba1e560b6db11439ff0f99103db35eec620d7
2 changes: 1 addition & 1 deletion sdk/evaluation/azure-ai-evaluation/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
### Features Added

### Breaking Changes
- The `parallel` parameter has been removed from composite evaluators: QAEvaluator, ContentSafetyChatEvaluator, and ContentSafetyMultimodalEvaluator. To control evaluator parallelism, you can now use the `_parallel` keyword argument, though please note that this private parameter may change in the future.
- The `parallel` parameter has been removed from composite evaluators: `QAEvaluator`, `ContentSafetyChatEvaluator`, and `ContentSafetyMultimodalEvaluator`. To control evaluator parallelism, you can now use the `_parallel` keyword argument, though please note that this private parameter may change in the future.

### Bugs Fixed

Expand Down