Skip to content

Conversation

@arun-rfai
Copy link
Collaborator

@arun-rfai arun-rfai commented Nov 20, 2025

Changes

  • added comment cells to gsm8k and scifact notebooks on openai api costs
  • downsampled data further in both notebooks to reduce cotsts
  • now api costs are $3 and $5 only
  • reasoning effort knob adjusted in gsm8k, reducing to 6 configs

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this change manually
  • I have tested this change in the following environments:
    • Local development
    • Docker environment
    • Other: _______________

Copy link
Collaborator

@chethan-rfai chethan-rfai left a comment

Choose a reason for hiding this comment

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

LGTM
This PR is expected to fix: #103

@arun-rfai arun-rfai merged commit b4b15e7 into main Nov 20, 2025
@arun-rfai arun-rfai deleted the feature/rag-openai-costs branch November 20, 2025 02:05
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.

3 participants