Skip to content
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

ChromaStorageConnector.get_all() takes 1 positional argument but 2 were given #747

Closed
clarkandrew opened this issue Jan 1, 2024 · 4 comments

Comments

@clarkandrew
Copy link

MemGPT cloned and installed locally on OSX.

There is a bug when trying to use persistent Chroma storage.

TypeError: ChromaStorageConnector.get_all() takes 1 positional argument but 2 were given

The issue is with this line in MemGPT/memgpt/memory.py

for passage in list(self.storage.get_all(limit)): 
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^

... meanwhile, the get_all method inside ChromaStorageConnector found in MemGPT/memgpt/connectors/chroma.py has no limit param:

def get_all(self) -> List[Passage]:
@quantumalchemy
Copy link

getting this always with using quick start / local chroma persistent -- anyone fix this?

@sarahwooders
Copy link
Collaborator

@quantumalchemy what version are you on, or did you install from source?

@quantumalchemy
Copy link

211 pip install -U pymemgpt
.. but I see there was a new rel 2 hours ago .. will try tomorrow

Copy link

github-actions bot commented Dec 6, 2024

This issue has been automatically closed due to 60 days of inactivity.

@github-actions github-actions bot closed this as completed Dec 6, 2024
carenthomas pushed a commit that referenced this issue Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants