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

Add script for experiments with ResNet50 #14

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Ambuj-Choudha
Copy link

Hi,

I have added the commands which I ran for ResNet50 experiments. These commands are consistent with TL-lib (i.e. without --no-pool flag).
Regarding the changes I made in pytorch code, I have not added those changes in this pull request because it really depends with which version is the person willing to run the repo code.

For more reference check the open issue thread (#13)
The issue is with ViT arch which is still open.

Thanks,
AC

@rangwani-harsh
Copy link
Contributor

Hi @Ambuj-Choudha,

It would be great if you could also add a file name cdan_sdat_tllib.py which contains all the changes which are required for tllib, which you had mentioned in #13.

Also please update the script based on the new file name.

Thanks
Harsh

@Ambuj-Choudha
Copy link
Author

Hey hi,

I hope I am not causing confusion.
Only for running the experiments within TL lib, this change is required:
x_s, labels_s = next(train_source_iter)[:2]
x_t, _ = next(train_target_iter)[:2]

but it doesn't cause any issue, in the SDAT repo. Only removing --no-pool flag is still neccesary.

Regards,
AC

@rangwani-harsh
Copy link
Contributor

Hi @Ambuj-Choudha,

Thanks for your PR, I will try to review this over the weekend, and then get back to you.

Thanks
Harsh

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

Successfully merging this pull request may close these issues.

2 participants