[PR #127] [MERGED] remove with_rate_limit from public api #135

Closed
opened 2026-02-16 00:18:18 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langchain-benchmarks/pull/127
Author: @eyurtsev
Created: 12/13/2023
Status: Merged
Merged: 12/13/2023
Merged by: @eyurtsev

Base: mainHead: eugene/fix_rate_limiter


📝 Commits (1)

📊 Changes

3 files changed (+2 additions, -4 deletions)

View changed files

📝 langchain_benchmarks/__init__.py (+1 -2)
📝 langchain_benchmarks/tool_usage/agents.py (+1 -1)
📝 tests/unit_tests/test_public_api.py (+0 -1)

📄 Description

Because it's not a runnable binding it breaks things like .bind etc, let's use
it internally but not expose to users


🔄 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/langchain-benchmarks/pull/127 **Author:** [@eyurtsev](https://github.com/eyurtsev) **Created:** 12/13/2023 **Status:** ✅ Merged **Merged:** 12/13/2023 **Merged by:** [@eyurtsev](https://github.com/eyurtsev) **Base:** `main` ← **Head:** `eugene/fix_rate_limiter` --- ### 📝 Commits (1) - [`f3f1239`](https://github.com/langchain-ai/langchain-benchmarks/commit/f3f12397291755a753d8f36201894309c60b70cc) x ### 📊 Changes **3 files changed** (+2 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `langchain_benchmarks/__init__.py` (+1 -2) 📝 `langchain_benchmarks/tool_usage/agents.py` (+1 -1) 📝 `tests/unit_tests/test_public_api.py` (+0 -1) </details> ### 📄 Description Because it's not a runnable binding it breaks things like .bind etc, let's use it internally but not expose to users --- <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-16 00:18:18 -05:00
yindo closed this issue 2026-02-16 00:18:18 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langchain-benchmarks#135