Skip to content

fix: update rank_llm import paths (rank_llm@70d8a8f)#57

Merged
bclavie merged 1 commit into
AnswerDotAI:mainfrom
hoorangyee:fix/update-rank_llm-import-paths
Feb 13, 2025
Merged

fix: update rank_llm import paths (rank_llm@70d8a8f)#57
bclavie merged 1 commit into
AnswerDotAI:mainfrom
hoorangyee:fix/update-rank_llm-import-paths

Conversation

@hoorangyee

@hoorangyee hoorangyee commented Feb 13, 2025

Copy link
Copy Markdown
Contributor

In rank_llm's commit 70d8a8f, reranker modules were moved to the listwise directory.

This PR updates the import paths accordingly.

Changed import paths:

  • vicuna_reranker.py -> listwise/vicuna_reranker.py
  • zephyr_reranker.py -> listwise/zephyr_reranker.py
  • rank_gpt.py -> listwise/rank_gpt.py

Before this fix, users would encounter the following error even when rank_llm was properly installed:

You don't have the necessary dependencies installed to use RankLLMRanker. 
Please install the necessary dependencies for RankLLMRanker by running pip install "rerankers[rankllm]" or pip install "rerankers[all]" to install the dependencies for all reranker types.

Changed import paths:
- vicuna_reranker.py -> listwise/vicuna_reranker.py
- zephyr_reranker.py -> listwise/zephyr_reranker.py  
- rank_gpt.py -> listwise/rank_gpt.py
@bclavie

bclavie commented Feb 13, 2025

Copy link
Copy Markdown
Collaborator

Thank you! Merging now.

@hoorangyee

Copy link
Copy Markdown
Contributor Author

Thanks for merging!

seanmacavaney pushed a commit to seanmacavaney/rerankers that referenced this pull request Jun 25, 2026
Changed import paths:
- vicuna_reranker.py -> listwise/vicuna_reranker.py
- zephyr_reranker.py -> listwise/zephyr_reranker.py  
- rank_gpt.py -> listwise/rank_gpt.py
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