CNES / demcompare

3D DEM comparison CNES tool
https://demcompare.readthedocs.io/
Apache License 2.0
30 stars 8 forks source link

reported shift #3

Closed sgascoin closed 1 year ago

sgascoin commented 2 years ago

I think that in the output log the reported "Planimetry 2D shift between DEM and REF" should be multiplied by the pixel size (or remove the meter unit) -> row : xx m -> col : yy m

duboise-cnes commented 2 years ago

Hello @sgascoin , thanks for the comment ! We are currently developing a refactor version of demcompare and I am reporting this remark in the list for next release. Don't hesitate if you have other remarks, following the internal refactoring, we will try to change/clean the global output of demcompare to be more adequate for users (command line output logs and report)

sgascoin commented 2 years ago

ok great! If I may.. A very useful feature would be the possibility to apply a mask to exclude some areas of the DEM in the coregistration process. Ideally this mask could be a shapefile or a raster in any reference system (demcompare would handle the reprojection to the DEM grid..).

duboise-cnes commented 1 year ago

Hello, the last demcompare version 0.5.0 (with new API) is published just now. The bug should be fixed in this version.

I report the proposal in future possible features, thanks for the comment.

Can you tell me if I can close this issue ? Hoping that this new version cope with your needs to compare DEM.

Cordially.