castorini / rank_llm

RankLLM is a Python toolkit for reproducible information retrieval research using rerankers, with a focus on listwise reranking.
http://rankllm.ai
Apache License 2.0
312 stars 39 forks source link

P1-separate scripts from packages #40

Closed sahel-sh closed 8 months ago

sahel-sh commented 8 months ago

several modules/files have both classes and main(), the main part should be split into a script file under script

wu-ming233 commented 8 months ago

I currently see:

with both classes and a main(). Are these all the intended files?

sahel-sh commented 8 months ago

Yes, any file that has a class and a main.

For all separation, the script should go under scripts with a similar name, like pyerini_retriver will have a script named run_pyserini_retriver unders scripts/ folder

sahel-sh commented 8 months ago

@richard5678 has taken this issue

Richard5678 commented 8 months ago

I am working on it

sahel-sh commented 8 months ago

Thank you @Richard5678 for letting this go to your colleague. Assigning to @wu-ming233 since he has completed the work

sahel-sh commented 8 months ago

Thank you @wu-ming233 for working on this