KaparthyReddy:fix/openai-embeddings-token-limit - Branch - langchain-ai/langchain

fix(openai): Respect 300k token limit for embeddings API requests

#33668Merged

Comparing

KaparthyReddy:fix/openai-embeddings-token-limit

(

93e6b72

) with

master

(

1bc8802

)

Untouched: 6

Skipped: 28

No successful run was found on master ( 9bd401a) during the generation of this report, so 1bc8802 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Benchmarks

Skipped (28)

Passed

test_init_time

libs/partners/openai/tests/unit_tests/chat_models/test_responses_standard.py::TestOpenAIResponses

12.1 ms*12.1 ms

test_init_time

libs/partners/openai/tests/unit_tests/chat_models/test_base_standard.py::TestOpenAIStandard

12.1 ms*12.1 ms

test_stream_time

libs/partners/openai/tests/integration_tests/chat_models/test_responses_standard.py::TestOpenAIResponses

857 ms*856.9 ms

test_stream_time

libs/partners/openai/tests/integration_tests/chat_models/test_responses_standard.py::TestOpenAIStandard

1.2 s*1.1 s

test_init_time

libs/partners/openai/tests/unit_tests/chat_models/test_azure_standard.py::TestOpenAIStandard

1.7 s*1.7 s

test_stream_time

libs/partners/openai/tests/integration_tests/chat_models/test_base_standard.py::TestOpenAIStandard

1.2 s*1.2 s

Commits

Click on a commit to change the comparison range

Base

master

1bc8802

+0.05%

revert extraneous changes

15e86cf

22 days ago

by ccurme

-0.08%

Merge branch 'master' into fix/openai-embeddings-token-limit

0735f85

15 days ago

by mdrxy

+0.06%

93e6b72

15 days ago

by mdrxy

Home Terms Privacy Docs