castorini / rank_llm

Repository for prompt-decoding using LLMs (GPT3.5, GPT4, Vicuna, and Zephyr)
http://rankllm.ai
Apache License 2.0
277 stars 36 forks source link

Add test coverage for pyserini_retriever #72

Closed sahel-sh closed 5 months ago

sahel-sh commented 5 months ago

@Richard5678 this is sample test that I talked about, PTAL! This file was a bit tricky since it had dependencies on pyserini that needed mocking. I expect most files to be easier than this. Getting mocking right, might be a bit tricky, in my experience chatGPT gets it mostly right, you just need to tweak it a little bit to get it work

sahel-sh commented 5 months ago

Merging this to unblock Richard. @ronakice if you have any comments, please leave it as issue(s) and I will address them.