Skip to content

feat: Add model kwargs to SentenceTransformersRanker#6627

Merged
sjrl merged 7 commits into
v1.xfrom
ranker-model-kwargs
Dec 22, 2023
Merged

feat: Add model kwargs to SentenceTransformersRanker#6627
sjrl merged 7 commits into
v1.xfrom
ranker-model-kwargs

Conversation

@sjrl
Copy link
Copy Markdown
Contributor

@sjrl sjrl commented Dec 22, 2023

Related Issues

  • fixes N/A

Proposed Changes:

Updates cross encoder init to take model kwargs so we can pass float16 as a loading option. Increases speed of ranking 40 docs by 4x when using float16.

How did you test it?

  • added unit tests

Notes for the reviewer

Checklist

@sjrl sjrl requested review from a team as code owners December 22, 2023 09:05
@sjrl sjrl requested review from dfokina and vblagoje and removed request for a team December 22, 2023 09:05
@sjrl sjrl changed the base branch from main to v1.x December 22, 2023 09:06
Comment thread haystack/nodes/ranker/sentence_transformers.py
@sjrl sjrl requested a review from vblagoje December 22, 2023 13:28
@sjrl
Copy link
Copy Markdown
Contributor Author

sjrl commented Dec 22, 2023

Hey @vblagoje I made the change as we discussed. Let me know what you think!

@vblagoje
Copy link
Copy Markdown
Member

This version is better; it is much more explicit about what is happening. Thank you @sjrl

@sjrl sjrl merged commit 5ff81c2 into v1.x Dec 22, 2023
@sjrl sjrl deleted the ranker-model-kwargs branch December 22, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic:tests type:documentation Improvements on the docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants