Skip to content

Frontend [AGE-1340]: Fix cache related issue that is breaking human a/b eval#2453

Merged
ardaerzin merged 1 commit intodevfrom
frontend-AGE-1430/fix-ab-eval-b-variant-cache
Feb 6, 2025
Merged

Frontend [AGE-1340]: Fix cache related issue that is breaking human a/b eval#2453
ardaerzin merged 1 commit intodevfrom
frontend-AGE-1430/fix-ab-eval-b-variant-cache

Conversation

@ardaerzin
Copy link
Copy Markdown
Contributor

  • cache was not being updated due to old transformer not taking into account the uri key.
  • also improved the case where we return cached data instead of performing a refetch

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
agenta ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 6, 2025 3:18pm
agenta-documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 6, 2025 3:18pm

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. Frontend labels Feb 6, 2025
@ardaerzin ardaerzin merged commit a9612de into dev Feb 6, 2025
aybruhm pushed a commit that referenced this pull request Feb 13, 2025
…-b-variant-cache

Frontend [AGE-1340]: Fix cache related issue that is breaking human a/b eval
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant