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

Documentation and simplification of the rotation matrix convention for pano #17

Open
gmaillet opened this issue Feb 8, 2016 · 3 comments

Comments

@gmaillet
Copy link

gmaillet commented Feb 8, 2016

The rotation matrix from the cameraMetaData json file is multiplied by three other matrix in sensor.js (itownsWay, Photogram_JMM, photgramme_image): why? Maybe for historical reasons related to the Stereopolis mobile mapping system.
It's time to clean that and to write the documentation (for example, explain the process for using rotation matrix from a MicMac process).

@mbredif
Copy link
Member

mbredif commented Feb 8, 2016

I confirm that these are legacy transforms to enforce undocumented frame conventions for Stereopolis datasets. I agree that these "convention matrixes" should go to the json metadata files to streamline the general case in Sensor.js.

@vpicavet
Copy link
Contributor

vpicavet commented Feb 9, 2016

Is this a blocker for 1.0 release or can it wait for 1.1 ?

@gmaillet
Copy link
Author

gmaillet commented Feb 9, 2016

It's not a blocker, just a good oportunity to release the 1.1 soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants