8000 Automatic outputs alignment via CODEM by pierotofy · Pull Request #1565 · OpenDroneMap/ODM · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Automatic outputs alignment via CODEM #1565

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

Merged
merged 15 commits into from
Dec 14, 2022
Merged

Conversation

pierotofy
Copy link
Member
@pierotofy pierotofy commented Dec 13, 2022

This PR adds the ability to automatically register a reconstruction (and resulting orthophotos, point clouds, 3D models, etc.) to a reference DSM (GeoTIFF, LAS or LAZ) by using CODEM (https://github.com/NCALM-UH/CODEM, credits to Craig Glennie and the group at CRREL).

The details of the method are available at https://github.com/NCALM-UH/CODEM/blob/main/docs/details.md

Usage:

  • Pass a file via --align <path-to-tif-or-las-laz>.
  • Place a align.tif, or align.laz or align.las in the project root (similar to GEO and GCP files).

Sample output report:

image

This could be particularly useful for repeat flights of the same area where automatic alignment allows for better change detection analysis, without needing GCPs.

Also fixes #1564

@pierotofy pierotofy mentioned this pull request Dec 13, 2022
@pierotofy
Copy link
Member Author

Implements #1557 via OpenDroneMap/OpenSfM@cca015d

@pierotofy
Copy link
Member Author

Passes test suite. Ready to go.

@pierotofy pierotofy merged commit 98ffcca into OpenDroneMap:master Dec 14, 2022
@smathermather
Copy link
Contributor

Woah. I wonder if some interaction between this approach and the alignment step in split merge might be meaningful.

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

Successfully merging this pull request may close these issues.

Possible Bug: Qhull precision error: Initial simplex is flat
2 participants
0