[PR #80] [MERGED] Remove catch for embedding failure #100

Closed
opened 2026-02-15 19:15:01 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/auto-evaluator/pull/80
Author: @rlancemartin
Created: 4/30/2023
Status: Merged
Merged: 4/30/2023
Merged by: @benisgold

Base: mainHead: remove_try_catch_for_tiktoken_error


📝 Commits (2)

📊 Changes

3 files changed (+3 additions, -14 deletions)

View changed files

📝 api/evaluator_app.py (+1 -8)
📝 nextjs/components/Sidebar.tsx (+0 -4)
📝 nextjs/public/testData/experiments.json (+2 -2)

📄 Description

Don't silently fix the failure by using a different choice of embeddings.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/auto-evaluator/pull/80 **Author:** [@rlancemartin](https://github.com/rlancemartin) **Created:** 4/30/2023 **Status:** ✅ Merged **Merged:** 4/30/2023 **Merged by:** [@benisgold](https://github.com/benisgold) **Base:** `main` ← **Head:** `remove_try_catch_for_tiktoken_error` --- ### 📝 Commits (2) - [`bee7d1e`](https://github.com/langchain-ai/auto-evaluator/commit/bee7d1e7a6e59552805870e493ab9e95c9607e46) Remove catch for embd failure - [`20b0aa0`](https://github.com/langchain-ai/auto-evaluator/commit/20b0aa0c8f9e0eaca3fee1ad1cf6bd0d7562a563) frontend ### 📊 Changes **3 files changed** (+3 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `api/evaluator_app.py` (+1 -8) 📝 `nextjs/components/Sidebar.tsx` (+0 -4) 📝 `nextjs/public/testData/experiments.json` (+2 -2) </details> ### 📄 Description Don't silently fix the failure by using a different choice of embeddings. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 19:15:01 -05:00
yindo closed this issue 2026-02-15 19:15:01 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/auto-evaluator#100