search

OpenDroneMap / ODM

A command line toolkit to generate maps, point clouds, 3D models and DEMs from drone, balloon or kite images. 📷
https://opendronemap.org
GNU Affero General Public License v3.0
4.81k stars 1.09k forks source link

Add PDF reports while using split-merge #1227

Open pierotofy opened 3 years ago

pierotofy commented 3 years ago

Currently ODM does not generate PDF reports when using split-merge. It could be possible to generate them by:

smathermather commented 3 years ago

I'd lean toward aggregate stats. Any reason to not have submodel stats as well? For larger datasets it could be more useful than global stats.

pierotofy commented 3 years ago

I agree to both. The bottom line is that the split-merge report would contain slightly different information compared to normal runs.

smathermather commented 3 years ago

Yup. Agreed.

YanNoun commented 3 years ago

@pierotofy opensfm/action/compute_statistics.py calls stats.py functions that works on a list of reconstructions.

pierotofy commented 3 years ago

Didn't know that. Thanks @YanNoun ! I'll try it out.

YanNoun commented 3 years ago

The save_heatmap/save_topview might be super slow with 100K/1M points, feel free to run some random index-based decimation.

pierotofy commented 3 years ago

I did notice some slowdowns with larger datasets; would this be a good idea for a PR to OpenSfM for the compute_statistics command? (I could add a --decimate flag)

YanNoun commented 3 years ago

Yup definitely, that would be great !

pierotofy commented 3 years ago

https://github.com/mapillary/OpenSfM/pull/695

I opted to allow a user to set a cap on the number of points using random sampling. Definitely helps keep runtime in check!