Caltech

[Prateek0xeo]

#104
@cregouby
I have referred the pytorch vision library for this dataloader it is using google drive to load the data
My PR has not passed any test yet. I'll perfect the PR in the coming weeks and try to make the dataloader work
ISSUE
Another issue that i am coming up with is the get_image_size.magick-image Util that is not working in my dataloader
which i am trying to use for converting the images to RGB
Q.
also should I make the PR on top of the commits i did on the Eurosat or the main repo of torchvision(I'll delete this PR and make a new one)

[cregouby]

Hello @Prateek0xeo

This P.R. cannot include the code change of the P.R. #122 ! They must be independant !

The branch you create for a new feature must be created from main, and not from your last branch.

To fix that, I would close that P.R. and before deleting your branch Prateek0xeo:add-caltech101-dataloader, recreate a brand new feature branch from main and import the files dedicated to caltech in it (via git checkout add-caltech101-dataloader -- dataset-caltech.R )

Then for the code in the file itself, please review #124 and apply it in here !

[Prateek0xeo]

@cregouby
Instead of deleting i have force pushed my commits on the branch and deleted the commits from the Eurosat dataloader Is it fine?

[cregouby]

Hello @Prateek0xeo
The scope of 5 files modified is OK, but the content is not fine at all. Please have a look at R/dataset-caltech.R.

[Prateek0xeo]

@cregouby I tried using the google drive to load the dataset but was running into problems as I couldn't scrap the confirmation token from the website so i am trying hugging face now. there is still some problem with the code. I'll see if i can fix that.

[Prateek0xeo]

ok so this new commit downloads the parquet file completely but still there are errors