Skip to content

(ts)view: miscellaneous updates #548

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Apr 17, 2021
Merged

(ts)view: miscellaneous updates #548

merged 5 commits into from
Apr 17, 2021

Conversation

yunjunz
Copy link
Member

@yunjunz yunjunz commented Apr 17, 2021

Description of proposed changes

  • view: support file in UTM projection for H3Py (ASF HyP3 Support #540) as described in ingest hyp3 #542.
  • more flexible ref-date support in reference_date.py and tsview.py
  • plot_network: correct labels for offset SNR
  • load_gbis: add hgt to output HDF5

Reminders

  • Pass Codacy code review (green)
  • Pass Circle CI local 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.

yunjunz added 5 commits April 17, 2021 13:22
+ load_gbis:
  - add hgt from .mat file to output HDF5 file since this height could be corrected from ellipsoid to geoid already, which is more suitable for geophysical interpretation.
  - do not use "height" to avoid confusion because it's pre-occupied by mintpy geometry file for FILE_TYPE by readfile.read_attribute()

+ asc_desc2horz_vert: API supp for matrix
+ utils/network.thres*_baseline(): support both date12_list format
+ plot_network: correct labels for offset SNR
+ reference_date: support HDF5 file as input for --ref-date option, in addition to existing input types

+ utils1.update_template_file(): bug fix when the exiting key value contains *, by treating it as a normal character, instead of wildcard
+ view.update_data_with_plot_inps():
   - support spatial referencing on the fly with the reference point out of the subset box, for time-series file only.
   - check ref_val before re-referencing and ignore maskedout or nan ref_val

+ (ts)view: pass print_msg to auto_adjust_colormap_lut_and_disp_limit()

+ tsview: remove _ in default outfile base

+ utils0: add polygon2mask() to convert a polygon into a binary mask file.

+ deramp: change mask_in from required to optional with default as None

+ smallbaseApp.plot_results(): move opt common to the front for better alignment in the printout msg

+ defaults/smallbaselineApp.cfg: fix typo for recommended

+ ts2vel: linalg.lstsq add default rcond to avoid warning msg
+ readfile.read_gdal_vrt():
   - add X/Y_FIRST/STEP for all coordinate types
   - add UTM_ZONE for file in UTM projection
   - add X/Y_UNIT in meters/degrees

+ prep_hyp3: use readfile.read_gdal_vrt() to replace add_geospatial_metadata() for less rerdundancy and the new UTM_ZONE attribute

+ prep_gmtsar: codacy suggestion

+ view: support file in UTM projection by default
   - support native matplotlib display in geo-coord via setting extent arg for files in UTM projection
   - remove the redundant and obsolete code block in plot_slice() for coord unit of 'm'
   - untie --coastline and --lalo-label option because cartopy seems to solve the previous default blank ticks issue
   - raise exception for --lalo-label for file in UTM projection
   - add code to auto determine the map projection for files with coord unit of deg and meter.

+ utils/plot.draw_scalebar(): add unit arg to support coordinate unit of meters

+ utils/arg_group.add_map_arg*(): comment out --proj option because we can not really plot the raster image in a map projection that is different from the native image data projection/transform using cartopy, at least not in the non-global scale
@yunjunz yunjunz merged commit 7adb3a1 into insarlab:main Apr 17, 2021
@yunjunz yunjunz added this to the UTM coordinate support milestone Aug 15, 2021
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