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

changeable resolution #72

Merged
merged 1 commit into from
Jan 7, 2024
Merged

changeable resolution #72

merged 1 commit into from
Jan 7, 2024

Conversation

shinning0821
Copy link
Collaborator

The size of the input images can now be changed. Using a smaller resolution can reduce the computational requirements of model training.

@WuJunde
Copy link
Collaborator

WuJunde commented Jan 7, 2024

nice work, ziyue 👍 I gave the code a quick look, and everything seems cool, so it's a green light for the merge from me ✅.

Just a small note: could you throw in a mention of your test case in your latter commits? Something like "training and testing success on ISIC" would be really cool. It helps us all to get where your code works and makes our further debugging a breeze.

Anyway, massive thanks for your contribution! 🌟

@WuJunde WuJunde merged commit 887a87d into MedicineToken:main Jan 7, 2024
zeinebBC pushed a commit to zeinebBC/Medical-SAM-Adapter that referenced this pull request Jan 16, 2024
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