-
Notifications
You must be signed in to change notification settings - Fork 888
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
[Executor] Fix flex flow metrices #3010
[Executor] Fix flex flow metrices #3010
Conversation
promptflow SDK CLI Azure E2E Test Result dev/linatang/fix-flex-flow-aggregation-metrices 4 files 4 suites 5m 2s ⏱️ Results for commit acbb9bc. ♻️ This comment has been updated with latest results. |
SDK CLI Global Config Test Result dev/linatang/fix-flex-flow-aggregation-metrices5 tests 5 ✅ 1m 15s ⏱️ Results for commit acbb9bc. ♻️ This comment has been updated with latest results. |
Executor Unit Test Result dev/linatang/fix-flex-flow-aggregation-metrices792 tests 792 ✅ 3m 44s ⏱️ Results for commit acbb9bc. ♻️ This comment has been updated with latest results. |
Executor E2E Test Result dev/linatang/fix-flex-flow-aggregation-metrices241 tests 236 ✅ 5m 10s ⏱️ Results for commit acbb9bc. ♻️ This comment has been updated with latest results. |
promptflow-core test result0 tests 0 ✅ 0s ⏱️ Results for commit acbb9bc. ♻️ This comment has been updated with latest results. |
SDK CLI Test Result dev/linatang/fix-flex-flow-aggregation-metrices 4 files 4 suites 58m 7s ⏱️ Results for commit acbb9bc. ♻️ This comment has been updated with latest results. |
Description
Currently, we always update metrices no matter whether the output of aggregation function is a dictionary or not, which is not reasonable. The expected output of aggregation function should be a dictionary, if not, we will skip metrices update and add a warning in logger.
All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines