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

Refactor stereopair metadata parsing and geometry plot for more flexibility #25

Open
bpurinton opened this issue Jul 8, 2024 · 1 comment

Comments

@bpurinton
Copy link
Contributor

On #24, the work done to extract scene metadata and plot the geometry (sky plots) was highly specific to WV scenes. Really, it should be written more flexibly for ASTER, HiRISE, etc., etc.

This is a hairy problem and requires a case-by-case study of metadata for each product. Likely this will need to be done iteratively at a later point after knocking out some of the other issues on the board here.

@bpurinton
Copy link
Contributor Author

bpurinton commented Aug 21, 2024

Actually, why re-invent the wheel for XML parsing? We should really just reuse the methods of ASP, like shown in this file:

https://github.com/NeoGeographyToolkit/StereoPipeline/blob/master/src/asp/Camera/RPC_XML.cc

All of the camera parsing is done already by ASP so we should extract relevant, useful details and output a report / table directly from ASP during processing. Then the code here can slurp that report which will be standardized for any given camera / sensor.

Alternatively to ASP outputting this report, can we call the methods of interest here?

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

1 participant