Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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