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
Fix bugs about OpenAI tokens #3067
Conversation
promptflow-tracing test result 12 files 12 suites 7m 17s ⏱️ Results for commit 22d64e8. ♻️ This comment has been updated with latest results. |
promptflow SDK CLI Azure E2E Test Result dev/linatang/fix-bugs-about-openai-tokens 4 files 4 suites 4m 9s ⏱️ Results for commit 22d64e8. ♻️ This comment has been updated with latest results. |
promptflow-core test result0 tests 0 ✅ 0s ⏱️ Results for commit 22d64e8. ♻️ This comment has been updated with latest results. |
SDK CLI Global Config Test Result dev/linatang/fix-bugs-about-openai-tokens6 tests 6 ✅ 1m 15s ⏱️ Results for commit 22d64e8. ♻️ This comment has been updated with latest results. |
Executor Unit Test Result dev/linatang/fix-bugs-about-openai-tokens792 tests 792 ✅ 3m 43s ⏱️ Results for commit 22d64e8. ♻️ This comment has been updated with latest results. |
Executor E2E Test Result dev/linatang/fix-bugs-about-openai-tokens243 tests 238 ✅ 5m 33s ⏱️ Results for commit 22d64e8. ♻️ This comment has been updated with latest results. |
SDK CLI Test Result dev/linatang/fix-bugs-about-openai-tokens 4 files 4 suites 1h 1m 6s ⏱️ Results for commit 22d64e8. ♻️ This comment has been updated with latest results. |
5b8b435
to
0c8d9ae
Compare
Description
Fix some bugs about OpenAI tokens:
BatchResult.system_metrices
, since we get the tokens from node run info in line result, but the flex flow doesn't have node run info, in this PR we get the tokens from flow run infos.OpenAIMetricsCalculator
.All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines