Hi @hardikdava 👋🏻 Is it ready for review?

@SkalskiP Yeah, it is ready for review.

@hardikdava I tried to run Notebook but it crashed. Is it up to date?

@SkalskiP Ohh, it is an issue of image downloading.

@SkalskiP Can you manually upload this image for testing? It should work. I just tested it.

@SkalskiP @hardikdava

raw URL good for wget : https://raw.githubusercontent.com/spmallick/learnopencv/master/Exploring-Slicing-Aided-Hyper-Inference/test_images/bird.jpg

Tip info : https://stackoverflow.com/questions/4604663/download-single-files-from-github/4605068#4605068

!wget 'https://raw.githubusercontent.com/spmallick/learnopencv/master/Exploring-Slicing-Aided-Hyper-Inference/test_images/bird.jpg'

from IPython.display import Image, display
display(Image('/content/bird.jpg'))

@SkalskiP It is updated now!, it is working. Thanks @onuralpszr

@SkalskiP It is updated now!, it is working. Thanks @onuralpszr

You're welcome :) I am glad to help

Hi @hardikdava 👋🏻 I'd add some unit tests to this PR. I can do that if ou don't have time.

Hi @hardikdava 👋🏻 I will work on a few unit tests before merging.

@SkalskiP Thanks for covering some tasks for me.

@hardikdava can I only ask one question? Why is _offset_generation producing offsets with 4 coordinates and not just two? I would assume we could move boxes with just x and y vectors.

@SkalskiP yeah, we can do it with just x and y.

I'll try to do some minor changes when I'll be working on tests.

@hardikdava slicer looks fantastic!

Looks really Amazing :)