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

Adjusts insights view to show calculation of cost #378

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

elijahbenizzy
Copy link
Contributor

@elijahbenizzy elijahbenizzy commented Sep 30, 2024

Adds explanation for LLM cost

image

Changes

  • adds explanation for LLM cost

How I tested this

  • manually

Notes

  • Only on individuals, not aggregates

Important

Enhances InsightsView.tsx to display detailed LLM cost breakdowns using a new Cost component with hover functionality.

  • Behavior:
    • Adds detailed cost breakdown for LLM operations in InsightsView.tsx.
    • Cost component now shows total cost and detailed breakdown on hover.
  • Functions:
    • Updates gatherCostAttributes to return detailed cost data including prompt and completion tokens and their respective costs.
    • Modifies RenderInsightValue and RenderIndividualValue in REGISTERED_INSIGHTS to use detailed cost data.
  • Misc:
    • Adds useState to manage hover state in Cost component.

This description was created by Ellipsis for cfbf3d4. It will automatically update as commits are pushed.

Copy link

github-actions bot commented Sep 30, 2024

A preview of cfbf3d4 is uploaded and can be seen here:

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

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/378/

@elijahbenizzy elijahbenizzy marked this pull request as ready for review September 30, 2024 18:13
@elijahbenizzy elijahbenizzy enabled auto-merge (rebase) September 30, 2024 18:14
Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to cfbf3d4 in 22 seconds

More details
  • Looked at 191 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_raXtnJVdqsXb4bQl


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@elijahbenizzy elijahbenizzy merged commit 8242b58 into main Sep 30, 2024
12 checks passed
@elijahbenizzy elijahbenizzy deleted the llm-cost-explain branch September 30, 2024 20:27
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