Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dataloader and updating the box #7

Closed
DanielaPlusPlus opened this issue Apr 4, 2022 · 3 comments
Closed

dataloader and updating the box #7

DanielaPlusPlus opened this issue Apr 4, 2022 · 3 comments

Comments

@DanielaPlusPlus
Copy link

DanielaPlusPlus commented Apr 4, 2022

Hi, great work! Thank you for sharing.
In the codes, training data is loaded after the contrastive crop and then is used for training through all epochs.
And for every location interval epoch, the box is updated.
The box is used for the contrastive crop in the data loader process to generate better sample pairs.
So how is the updating of the box used for generating better sample pairs?
I guess maybe we should load the training data every time we update the box, what do you think?

@xyupeng
Copy link
Owner

xyupeng commented Apr 5, 2022

Hi,
The key is not the dataloader, but the dataset.
The dataset holds the box information for every sample, which is used by ContrastiveCrop to generate better views. And the boxes are updated at some interval.

@DanielaPlusPlus
Copy link
Author

DanielaPlusPlus commented Apr 5, 2022

How will the updated box be used for the dataset?
In the codes, the dataset is loaded out of the epochs cycling with the initial box, the updating of the box is in the epochs cycling. How will the updating of the box affect the dataset?

@DanielaPlusPlus
Copy link
Author

Sorry, I got it. It's solved. The updating is used here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants