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

chroma func _similarity_search_with_relevance_scores missing "kwargs" parameter #6707

Closed
1 of 14 tasks
sudolong opened this issue Jun 25, 2023 · 2 comments
Closed
1 of 14 tasks
Labels
01 bug Confirmed bug needs work PRs that need more work

Comments

@sudolong
Copy link
Contributor

sudolong commented Jun 25, 2023

System Info

LangChain: v0.0.214

Who can help?

No response

Information

  • The official example notebooks/scripts
  • My own modified scripts

Related Components

  • LLMs/Chat Models
  • Embedding Models
  • Prompts / Prompt Templates / Prompt Selectors
  • Output Parsers
  • Document Loaders
  • Vector Stores / Retrievers
  • Memory
  • Agents / Agent Executors
  • Tools / Toolkits
  • Chains
  • Callbacks/Tracing
  • Async

Reproduction

https://github.com/hwchase17/langchain/blob/v0.0.214/langchain/vectorstores/base.py#L410
https://github.com/hwchase17/langchain/blob/v0.0.214/langchain/vectorstores/chroma.py#L225
https://github.com/hwchase17/langchain/blob/v0.0.214/langchain/vectorstores/chroma.py#L198

The filter option does not work when search_type is similarity_score_threshold

Expected behavior

work:

def _similarity_search_with_relevance_scores(
        self,
        query: str,
        k: int = 4,
        **kwargs: Any,
    ) -> List[Tuple[Document, float]]:
        return self.similarity_search_with_score(query, k, **kwargs)
@sudolong
Copy link
Contributor Author

@hwchase17 Please take a moment to take a look

@dosubot
Copy link

dosubot bot commented Sep 24, 2023

Hi, @sudolong! I'm Dosu, and I'm helping the LangChain team manage their backlog. I wanted to let you know that we are marking this issue as stale.

From what I understand, you opened this issue regarding a missing "kwargs" parameter in the chroma function _similarity_search_with_relevance_scores. You mentioned that the function should work with the "filter" option when the search_type is similarity_score_threshold. There has been recent activity with hwchase17 being requested to take a look at the issue.

Before we proceed, we would like to confirm if this issue is still relevant to the latest version of the LangChain repository. If it is, please let us know by commenting on the issue. Otherwise, feel free to close the issue yourself, or it will be automatically closed in 7 days.

Thank you for your understanding and contribution to the LangChain project!

@dosubot dosubot bot added the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Sep 24, 2023
@dosubot dosubot bot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 1, 2023
@dosubot dosubot bot removed the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Oct 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
01 bug Confirmed bug needs work PRs that need more work
Projects
None yet
Development

No branches or pull requests

1 participant