fix(metrics): avoid getting none in context_recall

[yuukidach]

Maybe this isssue #352 is talking about the same thing.

[shahules786]

Hi @yuukidach , thanks for the PR. This can be treated as a quick fix. IMO doing this might cause incorrect scores, Ideally on such edge cases score should be nan. Would you like to modify it?

[yuukidach]

@shahules786 Thanks for reviewing. I‘ve modified the code as required.

[shahules786]

LGTM

[yuukidach]

@shahules786 Hi, I just found out that CI failed during type check. These errors should have existed for some time and the CI configurations are not changed. Why did other relevant PRs not trigger this error before? I don't get it.

[yuukidach]

@shahules786 Hi, I just found out that CI failed during type check. These errors should have existed for some time and the CI configurations are not changed. Why did other relevant PRs not trigger this error before? I don't get it.

It seems pyright released a new version yesterday. Fixed to 1.1.338 in requirements/dev.txt

[shahules786]

Thank you @yuukidach I was wondering the same.

[jjmachan]

Hey @yuukidach
Thank you so much for helping us improve ragas with your PR ❤️
Now since it's Christmas and all we wanted to send you a postcard and a couple of stickers as our way of saying thank you for your contribution. If you are interested could you shoot an email at [email protected] and I'll tell you more?

Cheers 🙂
Jithin