jreyesr
commented
3 years ago Proposed fix:
- [x] Add another function in the Flight complete webhook, just before https://github.com/IS-AgroSmart/AgroSmart-Web/blob/e5928c976127e68bfcbc284ed81d5867331933d9/core/views.py#L287
- [x] Replicate files
color_relief.txtandhsv_merge.pyfrom https://github.com/OpenDroneMap/NodeODM/tree/18d6d218ed54a9705082651ae18a4af8a2fdf768/scripts, place them somewhere accesible to a Python function escaping to shell - [ ] In said function, call
os.systemand invoke the three commands on lines 31-33 of https://github.com/OpenDroneMap/NodeODM/commit/89584ee50bc4070613f442292bc3269f60de0764#diff-dd7d16b5de6fd001109c13820cc7525d6af18695f638bed53352a3aa01de85bb
App does no longer create the hillshaded DEM that is provided to the user for downloads and displayed in the flight report. Bug is due to a change upstream, in NodeODM, that no longer creates the hillshaded DEM during postprocessing.
See https://github.com/OpenDroneMap/NodeODM/commit/89584ee50bc4070613f442292bc3269f60de0764#diff-dd7d16b5de6fd001109c13820cc7525d6af18695f638bed53352a3aa01de85bb
Lines 22-37 create the file. They were nuked along some (now useless) tiling code. There's no mention of this change in the pull request (https://github.com/OpenDroneMap/NodeODM/pull/124)