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

allow HF model download, allow disabling tqdm bars #1059

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kylrth
Copy link

@kylrth kylrth commented Aug 15, 2024

The first commit fixes a problem where the layerwise reranker classes could not download models from HF because local_files_only=True was set.

closes #721

The second commit adds as progress keyword argument to all reranker compute_score methods, which allows the caller to activate/disable the tqdm progress bar. Some of these methods previously disabled the progress bar if the number of samples was less than the arbitrary value of 128. Now all methods will deactivate it by default if there is only one batch, but the user can choose to set it on or off regardless by passing progress=True or progress=False.

kylrth added 2 commits August 14, 2024 20:50
Also sets a reasonable default of showing the progress bar if more than one batch is processed.
@realJaydenCheng
Copy link

it is very useful for disabling the tqdm progress bar👍

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.

can't load reranker-v2-minicpm-layerwise model
2 participants