Skip to content
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 model costs #377

Closed
wants to merge 4 commits into from
Closed

Add model costs #377

wants to merge 4 commits into from

Conversation

skrawcz
Copy link
Contributor

@skrawcz skrawcz commented Sep 29, 2024

Adds model cost...

Changes

How I tested this

Notes

Checklist

  • PR has an informative and human-readable title (this will be pulled into the release notes)
  • Changes are limited to a single goal (no scope creep)
  • Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future TODOs are captured in comments
  • Project documentation has been updated if adding/changing functionality.

elijahbenizzy and others added 3 commits September 28, 2024 17:50
This is left with a placeholder for unhardcoding the values. Otherwise
it's fixed at 5.00/1m tokens, just to illustrate.
Adds raw data to be included in the UI for calculating
model costs.
This file is something we will need
to periodically update.
Copy link

github-actions bot commented Sep 29, 2024

A preview of 0307be2 is uploaded and can be seen here:

https://burr.dagworks.io/pull/377

Changes may take a few minutes to propagate. Since this is a preview of production, content with draft: true will not be rendered. The source is here: https://github.com/DAGWorks-Inc/burr/tree/gh-pages/pull/377/

@skrawcz
Copy link
Contributor Author

skrawcz commented Sep 29, 2024

closing this PR in favor of other one

@skrawcz skrawcz closed this Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants