Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5k
Add tif support#1274
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Add tif support#1274
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@kdionyso not opposed to adding, but one issue with tiff, aren't both .tif and .tiff equally common or do you find .tif is most used? |
kdionyso commentedJun 13, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Apologies for the very very tardy reply,@rwightman. I believe that ".tif" files are more common in the remote sensing/earth observation community and this is why I recommended adding that capability. There are possibly use cases for ".tiff" files as well, but to be honest I have not come across files with this extension in my line of work. Awesome work by the way :) |
VarIr commentedJul 6, 2022
metazool commentedJul 6, 2022
Sounds like it's safest to include both. It would be very useful to us |
rwightman commentedJul 7, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@VarIr@metazool@kdionyso
EDIT this will be merged later today / tomorrow |
This small modification allows .tif files to be read in and used for training/validation/testing. The modification is pretty small and I hope it can be incorporated into master. This will help the remote sensing community to use the timm package for model training. If necessary I can write a small test to demonstrate that it works with .tif files.