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

vwip interest points on top of aligned (i.e. map-projected) L and R images (probably just use L_sub / R_sub) #8

Open
bpurinton opened this issue Jun 5, 2024 · 3 comments
Labels

Comments

@bpurinton
Copy link
Contributor

No description provided.

@bpurinton
Copy link
Contributor Author

Not sure this is needed. There is already functionality to plot the .match files, and the ASP docs note:

.vwip files (intermediate files for creating .match files).

So the .vwip interest points are the original raw interest points on each image that are filtered down to the single .match points file of the interest points found in both image.

There might not be a strong use case at the moment of implementing vwip plotting functionality.

If we do want it, then port the functionality from: https://github.com/friedrichknuth/bare/blob/3e610858be44418fcf334d760e853ce7868be16d/bare/plot/plot.py#L189

@dshean
Copy link
Member

dshean commented Jul 15, 2024

OK, fine for now with other priorities.

The motivation here is to see the interest points on each of the original images (to check distribution), and then to assess the subset of those interest points was successfully matched (and distribution). If we only have the match points, and there are issues with match point number and distribution, we don't know if the issue was due to poor matches, or the fact that there were no interest points to begin with in a particular area of the input images (e.g., due to limited texture, clouds, water).

Matches are more important, and when things work well, that's all we need. It's the cases that fail where we want to check the original ip.

@bpurinton
Copy link
Contributor Author

Good points, I think it will be a nice feature to have, but as you said, other priorities first (i.e. going to start working on the ICESat plotting soon)

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

No branches or pull requests

2 participants