ragas icon indicating copy to clipboard operation
ragas copied to clipboard

Add tests for `llm.json_load`

Open jjmachan opened this issue 1 year ago • 4 comments

Describe the Feature bugs like the one @pberger514 caught in #593 might occur down the road too. We need tests for these sections of code. The catch here is that these are tests which need LLM calls so have to add a new Pytest mark for this

jjmachan avatar Feb 15 '24 08:02 jjmachan

Happy to pick this one up 😃

pberger514 avatar Feb 15 '24 14:02 pberger514

assigned - feel free to give it a go 🙌🏽

jjmachan avatar Feb 19 '24 06:02 jjmachan

we might need to colaborate to figure out the right pytest magic so that everything is easy to use from the CI.

I think we will have to test this for a couple of models like OpenAI, Bedrock, Llama2 and Mistral, have it part of the CI

jjmachan avatar Feb 19 '24 06:02 jjmachan

@jjmachan I put a first pass in a draft. Let me know your thoughts, I'm happy to expand to other models if you think this works.

pberger514 avatar Feb 25 '24 22:02 pberger514