From 905f43377b92c3fe98282240e67e2458efe9e533 Mon Sep 17 00:00:00 2001 From: Ankan Mahapatra <69568975+ankan2526@users.noreply.github.com> Date: Fri, 1 Nov 2024 00:07:41 +0530 Subject: [PATCH] Update word_document.py | Fixed metadata["source"] for web paths (#27220) The metadata["source"] value for the web paths was being set to temporary path (/tmp). Fixed it by creating a new variable self.original_file_path, which will store the original path. Thank you for contributing to LangChain! - [ ] **PR title**: "package: description" - Where "package" is whichever of langchain, community, core, etc. is being modified. Use "docs: ..." for purely docs changes, "templates: ..." for template changes, "infra: ..." for CI changes. - Example: "community: add foobar LLM" - [ ] **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! - [ ] **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. - [ ] **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. --- .../langchain_community/document_loaders/word_document.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libs/community/langchain_community/document_loaders/word_document.py b/libs/community/langchain_community/document_loaders/word_document.py index c7c50575cc7..bd2c4ae8f57 100644 --- a/libs/community/langchain_community/document_loaders/word_document.py +++ b/libs/community/langchain_community/document_loaders/word_document.py @@ -24,6 +24,7 @@ class Docx2txtLoader(BaseLoader, ABC): def __init__(self, file_path: Union[str, Path]): """Initialize with file path.""" self.file_path = str(file_path) + self.original_file_path = self.file_path if "~" in self.file_path: self.file_path = os.path.expanduser(self.file_path) @@ -55,7 +56,7 @@ class Docx2txtLoader(BaseLoader, ABC): return [ Document( page_content=docx2txt.process(self.file_path), - metadata={"source": self.file_path}, + metadata={"source": self.original_file_path}, ) ]