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

Skip to content

Conversation

@yunjunz
Copy link
Member

@yunjunz yunjunz commented Nov 10, 2020

Description of proposed changes

  • docs/demo_dataset: add ARIA example dataset on San Francisco prepared by @hfattahi using ARIA-tools with link to zenodo

  • add aria support in load_data.py by moving the prep_aria.main() call from smallbaselineApp.py to load_data.py. With this, the prep_aria.py appears to be the same as the other prep_*.py scripts (Spatial subset not working for ARIA data #469).

  • add 'mintpy.load.autoPath' support for aria.

  • add SanFranSenDT42 example dataset to test/test_smallbaselineApp.py for regular testing. Only local testing currently because downloading within circle ci every time would take quite some time.

Reminders

  • Pass Codacy code review (green)
  • Pass Circle CI test (green)
  • If modifying functionality, describe changes to function behavior and arguments in a comment below the function declaration.
  • If adding new functionality, add a detailed description to the documentation and/or an example.

+ update mintpy/data/input_files/SanFranSenDT42.txt: enable unwrapping error correction with bridging and disable network modification via interferogram index.

+ docs/demo_dataset: add ARIA section with zenodo link using the San Francisco dataset prepared by Heresh Fattahi using ARIA-tools

+ view.prep_slice(): update example comments regarding the cartopy projection
@yunjunz yunjunz requested review from dbekaert and hfattahi November 10, 2020 21:20
+ load_data:
   - move the call of prep_aria.main() from smallbaselineApp.py into load_data.py, so that prep_aria could appears to be the same as other prep_*.py in the user side, a.k.a. "load_data.py -t template file" works for ARIA as well.
   - support 'mintpy.load.autoPath' for aria
   - ignore failed prep_aria and continue load_data, becuase 1) the ARIA product could have been loaded into mintpy and deleted already and 2) the potential missing loaded HDF5 files will be reported laterward in smallbaselineApp.py, so it's okay to ignore it here.

+ utils1.update_template_file(): fix a bug when the option name contains the option value, i.e. mintpy.load.autoPath = auto.

+ add SanFranSenDT42.txt to the test_smallbaselinApp.py

+ mintpy/sh/plot_smallbaselineApp: plot ts_demErr.h5
@yunjunz yunjunz merged commit d05ad1e into insarlab:main Nov 13, 2020
@yunjunz yunjunz deleted the aria branch November 13, 2020 23:57
yunjunz added a commit to yunjunz/MintPy that referenced this pull request Jun 12, 2024
to show the full error msg, by reverting part of insarlab#472.
yunjunz added a commit that referenced this pull request Jun 12, 2024
* readfile: support Gamma SLC in complex32 format

+ utils.readfile: support Gamma slc in complex32 format
   - add scomplex to complex32 data type conversion
   - read_binary_file(): support .rslc/rmli file extension
   - read_complex_int16(): refactor to use cpx_band in str for a more flexible output data; revise the output from 2 matrices into 1 matrix
   - read_binary(): support complex32 while calling read_complex_int16(), since it's not supported in numpy.

+ utils.readfile.auto_no_data_value(): add geometrical offset files from alos2App with num of band = 1

+ view.viewer.plot(): set pixels with no data value to np.nan, instead of call np.ma.masked_where(), which seems over-complicated (compared with setting to nan) and I could not recall the reason anymore.

+ cli/prep_isce.py: set -f default from str to list, to be consistent with the non-default var type.

* circleci: remove ${HOME}/data dir after testing

attempt to free up the disk space, and to fix the circle CI testing failure.

* Revert "circleci: remove ${HOME}/data dir after testing"

This reverts commit 912298e.

* load_data: run prep_aria w/o try/except

to show the full error msg, by reverting part of #472.
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