LinasKo
LinasKo
Thanks @YHallouard ! I'll have a look at the start of next week
Hi @YHallouard Unforunately not. It's on my backlog, but honestly, supervision might steal all the time I have this year. I'd still like to give it a proper review, so...
Thanks @joaomarcoscrs! I'll get it merged tomorrow morning!
It's a mistake on our part. Don't worry about that one :wink:
On it, on it. This one will take time, likely the whole week. It can have consequences across the repo, so I must be very careful.
I've made some comments. I like your changes so far! It shows you've thought through the problem, in places like introducing of optional variable in `empty`, while avoiding the pitfall...
That looks good. For each contribution moving forward, the next step is to create a Colab notebook to test the changes. This should include generating Detections with and without metadata...
Hi @souhhmm 👋 My apologies, I missed your Colab. That is indeed an amazing set of tests. I'll take over from this point - there's a few checks I need...
Thank you for a fantastic contribution @souhhmm! Most take me hours to verify, but with your test suite and thoughtful changes, I barely had to do anything. Expect to see...
Hi @lfurtadoh 👋 Looking at the code, `MeanAveragePrecision` does not support OBB - only standard object detection boxes. Given we're expanding our OBB support, I'll open a feature request for...