-
Notifications
You must be signed in to change notification settings - Fork 20
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
Error processing odm_data_aukerman dataset #15
Comments
This error means that the EXIF metadata in each image doesn't have the appropriate tags in order for the software to determine the CCD sensor size in mm. In other words, make sure the FocalLength and 35mm FocalLength tags are set properly.. or if you'd like to send me the CCD sensor size width and height in mm I'll add this to the camera sensor database --- it should then automatically be detected. Thanks |
This is a from a Sony wx220:
|
Thanks. I will add to the camera database and figure out a more elegant solution to adding sensors without the proper EXIF metadata. I'll push an update tomorrow |
This camera is common and already discuss in MicMac forum. It needs to be like this :
Dji's users will have same problem. Workaround without writing in DicoCamera.xml is to set exif with MicMac command. Example : Another alternative is to write camera in MicMac-LocalChantierDescripteur.xml instead of DicoCamera.xml. Look also what they do in desktop python GUI here : https://github.com/micmacIGN/InterfaceCEREMA |
Thanks for the additional detail and it is correct -- except for the DJI part -- most DJI sensors have all the required EXIF metadata for MicMac to automatically compute the CCD sensor size. I think the best option will be to add command line parameters and option to allow the user to pass the CCD width / height in millimeters and generate the MicMac-LocalChantierDescripteur.xml accordingly. |
Ok didn't know about Dji. It just miss 35mm equivalent in WX220, okay ! |
This PR addresses and fixes the issue. 52add4e
Supporting multiple cameras and 3d models would be a future enhancement/feature.
Thanks |
When trying to process the Aukerman data set from odm (https://github.com/OpenDroneMap/odm_data_aukerman)
I get the following error:
The text was updated successfully, but these errors were encountered: