-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Add cost summary to client.py #812
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #812 +/- ##
===========================================
+ Coverage 28.00% 41.47% +13.46%
===========================================
Files 27 27
Lines 3524 3578 +54
Branches 803 860 +57
===========================================
+ Hits 987 1484 +497
+ Misses 2465 1954 -511
- Partials 72 140 +68
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Could you fix the test error? |
The location of the added notebook link is not right. I plan to fix that in another PR #851 |
Got it. Updated for test error. |
* init commit * add doc, notebook and test * fix test * update * update * update * update
* init commit * add doc, notebook and test * fix test * update * update * update * update
Why are these changes needed?
Add cost summary to client.py.
Including calculations for two types of cost:
Note: cost from completions that doesn't pass a filter is also included.
Minor fix: store cost before caching.
Related issue number
Checks