Image Path as keys for DetectionDataset

[hardikdava]

Description

sv.DetectionDatasets is using image_name as keys for Dict. This can be leading to create issues like same there can be possibility of having same name in multiple dataset. Also, saving image_path can be useful for not loading the whole image dataset but only load when necessary.

This PR is only redesigning the existing code.

[hardikdava]

@SkalskiP This is an initial PR is for lazy dataset loading and integration for yoloexplorer.

[hardikdava]

@capjamesg Please check it out if anything changes for autodistill.

[hardikdava]

@SkalskiP can we quickly merge this? Then we are good to go for yoloexplorer integration.

[SkalskiP]

@hardikdava merge it or release it? :) Because the release will probably come in 1-2 weeks. But merge can be today.

[hardikdava]

@SkalskiP let's merge it for now. I will make changes accordingly for initial integration.

[SkalskiP]

@hardikdava, we cant merge it. You made changes only in .from_* methods and not in .as_*. All methods that save annotations assume they operate on image names, not image paths, and try to join the path of the target directory with the source image path.

[hardikdava]

@SkalskiP noted.

[SkalskiP]

@hardikdava let me know when you'll have a version ready for tests 🙏🏻

[hardikdava]

Hey 👋 @SkalskiP , I have changed the required method. Let me know I have missed anything else.

[SkalskiP]

Hi, @hardikdava 👋🏻 do you maybe have some colab I could use for testing?

[SkalskiP]

Hi @hardikdava 👋🏻 I'm testing it now.

[SkalskiP]

Here is my colab I used for testing: https://colab.research.google.com/drive/1OstGePg0mKfe6tZr6dxAj2b4oBhlm657?usp=sharing

[SkalskiP]

Hi @hardikdava 👋🏻! Merging.