LinasKo
LinasKo
Yup, we'll need to look at this in the future.
Hi @PrakharJain1509 , Thank you very much! I'll have a look and review it tomorrow morning. Thank you for your contribution!
Looking at it right now :wink:
Hi @PrakharJain1509 👋 I can only see one commit at the moment, and that does not include everything you mentioned. Is there some code you haven't pushed yet? > Added...
Sure! The steps so far are correct. Did you write any code for the tests like you said? > Added integration tests for `MeanAveragePrecision` If you did, the code is...
Unit tests are welcome, but typically not required. However, since you wrote in the PR description that unit tests are included, I expect to see the code I'm reviewing to...
Hi @PrakharJain1509, I won't be able to help with testing, but can give some pointers regarding the Colab. If we can create the Colab that checks how the new functionality...
Hi @PrakharJain1509, Good work with the Colab. Stepping into a new repository and having to learn how everything is put together is tricky, and I can see you're putting good...
Test colab: https://colab.research.google.com/drive/1r5Z0No4cj0px3M_tbnG8mjk3JRjXsivF?usp=sharing
Merging - everything seems to be in order now. Thank you for the contribution @PrakharJain1509. Expect to see it included in the next supervision release! 🤝