forked from langchain-ai/langchain
-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/no core 0.3 #231
Closed
Closed
Feature/no core 0.3 #231
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…6316) - modify document links for create a tencent vectordb database instance. Co-authored-by: wlleiiwang <wlleiiwang@tencent.com> Co-authored-by: Erick Friis <erick@langchain.dev>
…i#26574) * Added rules `UPD006` now that Pydantic is v2+ --------- Co-authored-by: Eugene Yurtsev <eyurtsev@gmail.com>
…#26314) Co-authored-by: Erick Friis <erick@langchain.dev>
…chain-ai#26587) Co-authored-by: ccurme <chester.curme@gmail.com>
We recently renamed `MLflow Deployments Server` to `MLflow AI Gateway` in mlflow. This PR updates the relevant notebooks to use `MLflow AI gateway` --- Thank you for contributing to LangChain! - [x] **PR title**: "package: description" - Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..." for purely docs changes, "templates: ..." for template changes, "infra: ..." for CI changes. - Example: "community: add foobar LLM" - [x] **PR message**: ***Delete this entire checklist*** and replace with - **Description:** a description of the change - **Issue:** the issue # it fixes, if applicable - **Dependencies:** any dependencies required for this change - **Twitter handle:** if your PR gets announced, and you'd like a mention, we'll gladly shout you out! - [x] **Add tests and docs**: If you're adding a new integration, please include 1. a test for the integration, preferably unit tests that do not rely on network access, 2. an example notebook showing its use. It lives in `docs/docs/integrations` directory. - [x] **Lint and test**: Run `make format`, `make lint` and `make test` from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/ Additional guidelines: - Make sure optional dependencies are imported within a function. - Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests. - Most PRs should not touch more than one package. - Changes should be backwards compatible. - If you are adding something to community, do not re-import it in langchain. If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17. --------- Signed-off-by: harupy <17039389+harupy@users.noreply.github.com> Co-authored-by: Erick Friis <erick@langchain.dev>
…n-ai#26373) - add decorator for static methods Co-authored-by: Erick Friis <erick@langchain.dev>
updated OpenAI cost definitions according to the following: https://openai.com/api/pricing/ Co-authored-by: Erick Friis <erick@langchain.dev>
…angchain-ai#26423) - Add missing args Co-authored-by: Erick Friis <erick@langchain.dev>
…n-ai#26487) Thank you for contributing to LangChain! - [x] **PR title**: Fix typo in conda environment code block in rag.ipynb - In docs/tutorials/rag.ipynb Co-authored-by: Erick Friis <erick@langchain.dev>
…in-ai#26522) fix langchain-ai#26370 - langchain-ai#26370 `GoogleSpeechToTextLoader` is a deprecated method in `langchain_community.document_loaders.google_speech_to_text`. The new recommended usage is to use `SpeechToTextLoader` from `langchain_google_community`. When importing from `langchain_google_community`, use the name `SpeechToTextLoader` instead of the old `GoogleSpeechToTextLoader`. ![image](https://github.com/user-attachments/assets/3a8bd309-9858-4938-b7db-872f51b9542e) Co-authored-by: Erick Friis <erick@langchain.dev>
- **Description:** fixed a typo on amazon textract page Co-authored-by: Erick Friis <erick@langchain.dev>
…chain-ai#26328) - **Description:** the example to perform hybrid search with the Elasticsearch retriever is out of date - **Issue:** N/A - **Dependencies:** N/A Co-authored-by: Erick Friis <erick@langchain.dev>
- **Description:** typo in clickhouse vectorstore doc - **Issue:** langchain-ai#26597 - **Dependencies:** none - **Twitter handle:** sanjay920 Co-authored-by: Erick Friis <erick@langchain.dev>
Fixed typo `Unstrucutred`
… by ruff (langchain-ai#26608) Ruff doesn't know about the python version in `[tool.poetry.dependencies]`. It can get it from `project.requires-python`. Notes: * poetry seems to have issues getting the python constraints from `requires-python` and using `python` in per dependency constraints. So I had to duplicate the info. I will open an issue on poetry. * `inspect.isclass()` doesn't work correctly with `GenericAlias` (`list[...]`, `dict[..., ...]`) on Python <3.11 so I added some `not isinstance(type, GenericAlias)` checks: Python 3.11 ```pycon >>> import inspect >>> inspect.isclass(list) True >>> inspect.isclass(list[str]) False ``` Python 3.9 ```pycon >>> import inspect >>> inspect.isclass(list) True >>> inspect.isclass(list[str]) True ``` Co-authored-by: Eugene Yurtsev <eyurtsev@gmail.com>
makes core incompatible with uv installs
tested on one of the replacement cases and seems to work! ![ScreenShot 2024-09-18 at 02 02 43PM](https://github.com/user-attachments/assets/7170975a-2542-43ed-a203-d4126c6a2c81)
…er as agent tool (langchain-ai#26181) Thank you for contributing to LangChain! - [x] **PR title**: "package: description" - Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..." for purely docs changes, "templates: ..." for template changes, "infra: ..." for CI changes. - Example: "community: add foobar LLM" Added search options for BoxRetriever and added documentation to demonstrate how to use BoxRetriever as an agent tool - @BoxPlatform - [x] **Add tests and docs**: If you're adding a new integration, please include 1. a test for the integration, preferably unit tests that do not rely on network access, 2. an example notebook showing its use. It lives in `docs/docs/integrations` directory. - [x] **Lint and test**: Run `make format`, `make lint` and `make test` from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/ Additional guidelines: - Make sure optional dependencies are imported within a function. - Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests. - Most PRs should not touch more than one package. - Changes should be backwards compatible. - If you are adding something to community, do not re-import it in langchain. If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17.
Co-authored-by: Erick Friis <erick@langchain.dev>
Fixes langchain-ai#26212: replaced the raw string with backslashes. Alternative: raw-stringif the full docstring. --------- Co-authored-by: Erick Friis <erickfriis@gmail.com>
…ngchain-ai#25970) Co-authored-by: Erick Friis <erick@langchain.dev>
Public classes/functions are not renamed and rule is ignored for them. Co-authored-by: Erick Friis <erick@langchain.dev>
Missing "is" in a sentence about the Tool usage.
…ibute and prevent context loss (langchain-ai#26885) ## Description This PR fixes the context loss issue in `AsyncCallbackManager`, specifically in `on_llm_start` and `on_chat_model_start` methods. It properly honors the `run_inline` attribute of callback handlers, preventing race conditions and ordering issues. Key changes: 1. Separate handlers into inline and non-inline groups. 2. Execute inline handlers sequentially for each prompt. 3. Execute non-inline handlers concurrently across all prompts. 4. Preserve context for stateful handlers. 5. Maintain performance benefits for non-inline handlers. **These changes are implemented in `AsyncCallbackManager` rather than `ahandle_event` because the issue occurs at the prompt and message_list levels, not within individual events.** ## Testing - Test case implemented in langchain-ai#26857 now passes, verifying execution order for inline handlers. ## Related Issues - Fixes issue discussed in langchain-ai#23909 ## Dependencies No new dependencies are required. --- @eyurtsev: This PR implements the discussed changes to respect `run_inline` in `AsyncCallbackManager`. Please review and advise on any needed changes. Twitter handle: @parambharat --------- Co-authored-by: Eugene Yurtsev <eyurtsev@gmail.com>
Co-authored-by: Eugene Yurtsev <eyurtsev@gmail.com>
Fixed tutorial URL since earlier Tutorial URL was pointing to usecase age which does not have any detail it should redirect to correct URL page
…7104) **Description:** Fix reranking for empty lists **Issue:** ``` ValueError: not enough values to unpack (expected 3, got 0) documents, scores, vectors = map(list, zip(*docs)) File langchain_community/vectorstores/azuresearch.py", line 1680, in _reorder_results_with_maximal_marginal_relevance ``` Co-authored-by: Oleksii Pokotylo <oleksii.pokotylo@pwc.com>
All auto-fixes Co-authored-by: Erick Friis <erick@langchain.dev>
Adds - ASYNC - COM - DJ - EXE - FLY - FURB - ICN - INT - LOG - NPY - PD - Q - RSE - SLOT - T10 - TID - YTT Co-authored-by: Erick Friis <erick@langchain.dev>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Спасибо за помощь в развитии GigaChain!
При создании PR придерживайтесь простых правил: