Skip to content

Hotfix: Chroma does not raise NotEnoughElementsException anymore#334

Merged
ogabrielluiz merged 7 commits into
mainfrom
fix_chroma
May 23, 2023
Merged

Hotfix: Chroma does not raise NotEnoughElementsException anymore#334
ogabrielluiz merged 7 commits into
mainfrom
fix_chroma

Conversation

@ogabrielluiz
Copy link
Copy Markdown
Contributor

This error occurred if the number of documents in the index was lower than the number of documents requests.

deepankarm and others added 6 commits May 23, 2023 20:15
….py and test_chains_template.py

The display_name field was added to the LLM object in the test_agents_template.py and test_chains_template.py files. This field is used to display the name of the LLM in the UI.
…ption handling

Chroma implemented a fix for this so this exception won't be thrown anymore.
@ogabrielluiz ogabrielluiz added the Release Label to be set only on release PR label May 23, 2023
🔥 chore(test.yml): remove Python 3.11 from the matrix
Python 3.8 and 3.11 were removed from the matrix in lint.yml to reduce the number of Python versions being tested. Python 3.11 was removed from the matrix in test.yml as it is not yet released and therefore not supported.
Chroma uses onnxruntime which does not support 3.11 as of yet. This will be added back in once onnxruntime supports 3.11.
@ogabrielluiz ogabrielluiz merged commit d24404e into main May 23, 2023
@ogabrielluiz ogabrielluiz deleted the fix_chroma branch June 7, 2023 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release Label to be set only on release PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants