Mosaic, fix repo rename error, fix buffering issues #5
Conversation
Nice! Does this close #1 ? |
| else: | ||
| # process all at once, no tiling | ||
| dem_mosaic_opt.extend(['-o',outfn]) | ||
| out = run_cmd('dem_mosaic',dem_mosaic_args+dem_mosaic_opt) |
There was a problem hiding this comment.
TODO for later: look into executing this directly from Python rather than run_cmd. Should probably add asp and pdal as dependencies
Line 23 in 0b4803e
There was a problem hiding this comment.
yes, the issue is that asp does not have python API :( For everything else we will do the operations from within python!
Hi Scott, this addresses #1 partially. We do the mosaicking, but the reprojection step remains. I will contact you and David about that workflow later today/tomorrow since we need to check if the EPT files have heights relative to geoid or ellipsoid on our end. I will share a notebook; we can integrate it into the CLI workflow once we agree on the steps. |
Co-authored-by: Scott Henderson <scottyhq@gmail.com>
This PR introduces the ability to mosaic the output tiles into a final seamless composite.
While updating the code, I found issues in installation which was introduced due to not changing the repository name at some places while changing it at other places a few weeks ago. I also noticed that the buffering operation was not performed properly. I put fixes to both these issues. Finally, I updated the readme to reflect these changes.
Cheers,
Shashank