The latest updates on your projects. Learn more about Vercel for Git ↗︎

This last commit adapts the Cassandra Vector Store to the newest cassIO library, which introduces a few improvements (including interface changes). Also the delete method is implemented.
Moreover, an important non-deterministic source of trouble is identified and healed in the associated integration test (related to the MMR search test). There, test coverage is also extended to the delete methods.

Sorry about the poetry.lock conflict :(

@rlancemartin could you take a look at this Cassandra followup?

Thanks! Same as 6671, can you please rebase on master, resolve merge conflict in pyproject.toml and then re-generate poetry.lock with poetry lock --no-update? I can also take this (I'm already doing it for the other PR).

One other note: are there any conflicts between changes here and 6771 that we can resolve early (e.g., maybe they could be tested together by stacking the PRs).

Rebased. But when trying to run tests I get

$ poetry run pytest tests/integration_tests/vectorstores/test_cassandra.py 
Warning: Found deprecated key 'default' or 'secondary' in pyproject.toml configuration for source azure-sdk-dev. Please provide the key 'priority' instead. Accepted values are: 'default', 'primary', 'secondary', 'supplemental', 'explicit'.
Warning: Found deprecated priority 'secondary' for source 'azure-sdk-dev' in pyproject.toml. Consider changing the priority to one of the non-deprecated values: 'default', 'primary', 'supplemental', 'explicit'.
Warning: In a future version of Poetry, PyPI will be disabled automatically if at least one custom source is configured with another priority than 'explicit'. In order to avoid a breaking change and make your pyproject.toml forward compatible, add PyPI explicitly via 'poetry source add pypi'. By the way, this has the advantage that you can set the priority of PyPI as with any other source.
ImportError while loading conftest '/home/jonathan/Projects/langchain/tests/integration_tests/vectorstores/conftest.py'.
tests/integration_tests/vectorstores/conftest.py:7: in <module>
    from langchain.document_loaders import TextLoader
langchain/__init__.py:6: in <module>
    from langchain.agents import MRKLChain, ReActChain, SelfAskWithSearchChain
langchain/agents/__init__.py:2: in <module>
    from langchain.agents.agent import (
langchain/agents/agent.py:16: in <module>
    from langchain.agents.tools import InvalidTool
langchain/agents/tools.py:4: in <module>
    from langchain.callbacks.manager import (
langchain/callbacks/__init__.py:11: in <module>
    from langchain.callbacks.manager import (
langchain/callbacks/manager.py:35: in <module>
    from langchain.callbacks.tracers.langchain import LangChainTracer
langchain/callbacks/tracers/__init__.py:3: in <module>
    from langchain.callbacks.tracers.langchain import LangChainTracer
langchain/callbacks/tracers/langchain.py:11: in <module>
    from langchainplus_sdk import LangChainPlusClient
E   ModuleNotFoundError: No module named 'langchainplus_sdk'

Am I doing it wrong?

Test problem was due to me not re-running poetry install

Test problem was due to me not re-running poetry install

Glad you figured it out! I just kicked off tests.

@jbellis please run make format to resolve Lint error.

@rlancemartin Hello, the linter/code style should be happy now :)

The repeated conflict with poetry.lock is odd. My local master was up-to-date after 6728 went it, but I still saw this merge conflict reported in the PR after re-generating poetry.lock. It looks like you might be creating a new branch? Tag me on that and we should be able to get this in shortly.

@rlancemartin Sorry, I might have inadvertently done something untoward.
I wanted to sync my fork to upstream (including this branch), saw conflicts in the github UI and went with a force reset of upstream. That was a few minutes ago.
I think I zapped away the very content of this PR (looking at the code).

No problem. You should have the branch locally, and can force push your local branch to recover all.

Thank you! I actually did a git reset --hard on my pc but @jbellis will do the forcepush in a few minutes. Sorry about this and thank you for helping!

I pushed it back to where I left it yesterday after the reformat. Doesn't look like that is showing up yet on this PR page but I confirmed that the branch at https://github.com/hemidactylus/langchain/tree/SL-cassandra-vectorstore-2 is where it should be.

@rlancemartin perhaps the PR should be re-opened somehow? I see "no new commits" so I cannot reopen with this comment (yet the branch is restored).

@rlancemartin perhaps the PR should be re-opened somehow? I see "no new commits" so I cannot reopen with this comment (yet the branch is restored).

Try git push hemidactylus SL-cassandra-vectorstore-2

I get permission denied when I try to push from my local branch now (this worked previously) -

git push -f hemidactylus SL-cassandra-vectorstore-2
Enumerating objects: 415, done.
Counting objects: 100% (354/354), done.
Delta compression using up to 8 threads
Compressing objects: 100% (190/190), done.
Writing objects: 100% (263/263), 170.37 KiB | 14.20 MiB/s, done.
Total 263 (delta 172), reused 130 (delta 66), pack-reused 0
remote: Resolving deltas: 100% (172/172), completed with 67 local objects.
To github.com:hemidactylus/langchain.git
 ! [remote rejected]   SL-cassandra-vectorstore-2 -> SL-cassandra-vectorstore-2 (permission denied)
error: failed to push some refs to 'github.com:hemidactylus/langchain.git'

Can you re-open the PR?

If not, you should be able to simply open a new PR. If you do that, you probably want to change the branch name.

Thank you, I went with the "new PR" option (#7017). Thank you again!