Skip to content

mintpy.geocode.laloStep: support single value input#1075

Merged
yunjunz merged 1 commit intoinsarlab:mainfrom
yunjunz:geocode
Aug 23, 2023
Merged

mintpy.geocode.laloStep: support single value input#1075
yunjunz merged 1 commit intoinsarlab:mainfrom
yunjunz:geocode

Conversation

@yunjunz
Copy link
Member

@yunjunz yunjunz commented Aug 23, 2023

Description of proposed changes

  • geocode: support single input for the mintpy.geocode.laloStep option, as it is commonly used; and convert it into a list of two internally in the code.

  • smallbaselineApp.run_load_data(): tune the auto-skipping, by not adding the metadata from the custom template to the geometry HDF5 file, to avoid updating its file modification time.

  • timeseries_rms: auto adjust figure width for long TS

  • utils.utils1.spatial_average():

    • always read the output from the text file, which is in 1e-4 precision, to ensure consistency.
    • use the dataset MODIFICATION_TIME for ifgramStack file, for a more representative run/skip checking.

Reminders

  • Pass Pre-commit check (green)
  • Pass Codacy code review (green)
  • Pass Circle CI test (green)
  • Make sure that your code follows our style. Use the other functions/files as a basis.
  • If modifying functionality, describe changes to function behavior and arguments in a comment below the function declaration.

+ geocode: support single input for the "mintpy.geocode.laloStep" option, as it is commonly used; and convert it into a list of two internally in the code.

+ smallbaselineApp.run_load_data(): tune the auto-skipping, by not adding the metadata from custom template ot the geometry HDF5 file, to avoid updating its file modification time.

+ timeseries_rms: auto adjust figure width for long TS

+ utils.utils1.spatial_average():
   - always read the output from text file, which is in 1e-4 precision, to ensure consistency.
   - use the dataset MODIFICATION_TIME for ifgramStack file, for more representative run/skip checking.
@yunjunz yunjunz merged commit 136e67e into insarlab:main Aug 23, 2023
@yunjunz yunjunz deleted the geocode branch August 23, 2023 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant