Skip to content

Add UI elements for bandits#4069

Merged
virajmehta merged 208 commits intomainfrom
alan/bandits-viz-ui
Oct 23, 2025
Merged

Add UI elements for bandits#4069
virajmehta merged 208 commits intomainfrom
alan/bandits-viz-ui

Conversation

@amishler
Copy link
Member

@amishler amishler commented Oct 20, 2025

Important

Add UI components and backend logic for bandit experimentation, including feedback timeseries and optimal probability calculations.

  • UI Components:
    • Add FeedbackSamplesTimeseries component in FeedbackSamplesTimeseries.tsx to display cumulative feedback counts over time.
    • Update ExperimentationPieChart in PieChart.tsx to use memo for optimization.
    • Add TimeGranularitySelector in TimeGranularitySelector.tsx for selecting time granularity.
  • Experimentation:
    • Implement computeTrackAndStopOptimalProbabilities in experimentation.server.ts for calculating optimal sampling probabilities.
    • Update FunctionExperimentation.tsx to use optimal probabilities for track_and_stop experimentation.
  • Data Handling:
    • Modify route.tsx to fetch feedback timeseries and compute optimal probabilities.
    • Add jaro_winkler_similarity_feedback.jsonl to fixtures and update check-fixtures.sh and load_fixtures.sh to include it.
  • Testing:
    • Update integration.test.ts to include environment variable TENSORZERO_POSTGRES_URL for tests.

This description was created by Ellipsis for bca637d. You can customize this summary. It will automatically update as commits are pushed.

virajmehta and others added 30 commits September 26, 2025 11:23
Two constraint bugs:
- For the simplex constraint, had 1s everywhere instead of just at indices corresponding to the weights
- Signs were backwards in the SOCP constraints
@amishler
Copy link
Member Author

@codex review

@chatgpt-codex-connector
Copy link

To use Codex here, create a Codex account and connect to github.

@virajmehta
Copy link
Member

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@virajmehta virajmehta enabled auto-merge October 23, 2025 12:14
@virajmehta virajmehta added this pull request to the merge queue Oct 23, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 23, 2025
@virajmehta virajmehta added this pull request to the merge queue Oct 23, 2025
Merged via the queue into main with commit 64fca4d Oct 23, 2025
40 checks passed
@virajmehta virajmehta deleted the alan/bandits-viz-ui branch October 23, 2025 13:34
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