Skip to content

fix: issue with SentenceTransformer asking for HF token but not manda… - #144

Merged
lfnothias merged 1 commit into
mainfrom
fix_hf_token_issue
Apr 17, 2026
Merged

fix: issue with SentenceTransformer asking for HF token but not manda…#144
lfnothias merged 1 commit into
mainfrom
fix_hf_token_issue

Conversation

@Fosowl

@Fosowl Fosowl commented Apr 17, 2026

Copy link
Copy Markdown
Member

SentenceTransformer asking for HF token but not mandatory, so pass token=false

self.model = SentenceTransformer("all-MiniLM-L6-v2", token=False)

New users would get the issue unless they have a HF token despite not being necessary. To merge quickly

Contribution checklist

Please confirm the following before requesting review:

  • [ x] I have read CONTRIBUTING.md, docs/licensing-notes.md, and the repository license information (LICENSE, NOTICE) for this repository (Apache License 2.0).
  • [x ] I have read docs/cla-process.md and understand the CLA workflow for contributions.
  • [x ] I confirm that I have the legal right to submit this contribution.
  • [x ] I have signed the required short Individual Contributor Agreement, or I will complete it if requested.
  • [ x] If this contribution is made in the course of employment or under institutional IP rules, I understand that maintainers may also request the optional employer authorization.
  • [x ] I am not knowingly submitting code that is incompatible with this repository’s Apache 2.0 licensing terms and CLA requirements.

@lfnothias
lfnothias merged commit c9bbf07 into main Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants