Improvements

[BTOdell]

No description provided.

[BTOdell]

I just fixed a bug in the NMS algorithm. The old algorithm implementation was producing results like:

and the new implementation is producing results like:

Sorry the uploaded image quality is poor, but the bounding boxes towards the top of the screen are heavily duplicated and overlapping. The NMS algorithm should be cleaning these up but were not.

[Smirkey]

thanks a lot for the changes !

I think it won't be practical to handle this in a single PR (moving from owned arrays to views)
I changed the base branch to a dev branch

[BTOdell]

I only opened this PR to provide a reference into my fork. I don't plan on making further changes to my fork to fix the CI errors. I encourage you to just use my fork as a reference to make the changes across the entire project. My fork only has made the changes for a few select functions that I needed to use.

[BTOdell]

I'd be happy to answer any questions about why I made certain changes. I haven't run any benchmarks on the new code, so for all I know my implementation could be slower than yours. I was going for correctness as top priority.

[Smirkey]

you're right !
As for speed, using views is the way to go and I believe that it'll bring significant speedups specifically in python
there are automated benchmarks in the CI that will run when the tests will pass.

Thanks you very much for this work, I'll make the other necessary changes in the feature branch I created