MintPy icon indicating copy to clipboard operation
MintPy copied to clipboard

specify display unit for save_kmz_timeseries.py

Open pbrotoisworo opened this issue 3 years ago • 2 comments

Description of the desired feature

Can we add a parameter to change the display unit for save_kmz_timeseries.py. I notice that save_kmz.py has an argument for unit but I do not see the same option for save_kmz_timeseries.py. It is hard coded to be centimeters only.

Is your feature request related to a problem? Please describe

Describe the solution you'd like Add display unit argument in save_kmz_timeseries.py

Describe alternatives you have considered

Additional context

Are you willing to help implement and maintain this feature?

  • [x] Yes
  • [ ] No

pbrotoisworo avatar Sep 10 '22 07:09 pbrotoisworo

HI @pbrotoisworo, this sounds good to me. Please feel free to go ahead and implement it.

As you may have seem that save_kmz_timeiseries is much complicated than save_kmz, so please ensure all the units of velocties and time-series displacements are converted. And don't hesitate to let me know if you have questions.

yunjunz avatar Sep 11 '22 21:09 yunjunz

The default units of "cm" for save_kmz.py -v should be better specified in the help. I don't run it often, and I keep forgetting that it uses centimeters while the regular view.py -v units are meters and it is not obvious in the help usage.

EJFielding avatar Jan 19 '23 17:01 EJFielding