Skip to content

Added better way of storing pricing information (CAN-376) - #123

Draft
Saaketh0 wants to merge 2 commits into
CAN-356-Fixing-Claude-GPT-Costsfrom
CAN-376-Fixing-LLM-Token-Pricing
Draft

Saaketh0 wants to merge 2 commits into
CAN-356-Fixing-Claude-GPT-Costsfrom
CAN-376-Fixing-LLM-Token-Pricing

Conversation

@Saaketh0

Copy link
Copy Markdown
Collaborator

Previously, we had the pricing data in a SQLite DB, which just added complexity. Now we have a .yaml file that stores all of this which gets put into memory at init, due to the small size of the .yaml.

@Saaketh0 Saaketh0 self-assigned this Sep 17, 2026
@Saaketh0
Saaketh0 added this pull request to stack #124 September 17, 2026 01:01
@Saaketh0 Saaketh0 added the enhancement New feature or request label Sep 17, 2026
@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Saaketh0

Copy link
Copy Markdown
Collaborator Author

Note, should probably fix with when the OTel Metrics and Logs PR gets merged in, since those PR's change the OTLP process alot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant