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

Question for the dataset.py #7

Closed
halbielee opened this issue Jul 30, 2021 · 3 comments
Closed

Question for the dataset.py #7

halbielee opened this issue Jul 30, 2021 · 3 comments

Comments

@halbielee
Copy link

Hello, thanks for sharing your excellent work!

I have a question about your code, especially coco_train_dataset.py.

# Invariance transform.
image1 = self.transform_inv(index, image, 0)
image1 = self.transform_tensor(image)
if self.mode == 'baseline_train':
return (image1, )
image2 = self.transform_inv(index, image, 1)
image2 = TF.resize(image2, self.res1, Image.BILINEAR)
image2 = self.transform_tensor(image2)

When applying the transform_tensor function to the image, it applies to the variable image not image1.

Is there any issue for this?

Thanks!

@janghyuncho
Copy link
Owner

janghyuncho commented Jul 30, 2021

Hi halbielee,

Thank you for your interest in our work! I am happy to help you.

Indeed, you are absolutely correct that it appears to be my mistake during refactoring. The code for the result from the paper did not have this bug (only during refactoring). However, we did try this version before and it makes very small difference; in fact, many contrastive learning papers choose this way (contrasting between x <-> T(x) instead of T1(x) <-> T2(x)).

I fixed the code just now.

Thank you!

Jang Hyun Cho

@janghyuncho
Copy link
Owner

janghyuncho commented Jul 31, 2021

Hi halbielee,

Here is the training log of PiCIE with the updated code.

ACC - All: 51.5297
mIOU - All: 14.7917
ACC - Thing: 76.6696
mIOU - Thing: 18.1085
ACC - Stuff: 64.4714
mIOU - Stuff: 26.3870

Best,

Jang Hyun Cho

@halbielee
Copy link
Author

Thank you for the quick and kind reply :)

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