Skip to content

read_attr(): UNIT bugfix for dsName with date info#630

Merged
yunjunz merged 1 commit intoinsarlab:mainfrom
yunjunz:read
Aug 5, 2021
Merged

read_attr(): UNIT bugfix for dsName with date info#630
yunjunz merged 1 commit intoinsarlab:mainfrom
yunjunz:read

Conversation

@yunjunz
Copy link
Member

@yunjunz yunjunz commented Aug 5, 2021

Description of proposed changes

  • utils.readfile.read_attribute(): fix a bug of grabbing the UNIT for HDF5 dataset, if datasetName is specified with date info as suffix. This only affects the unit display in the colorbar during plot.
  • utils.utils1.get_residual_std/rms(): use np.float32 to replace np.float to avoid numpy deprecation warning
  • view: print WARNING msg regardless of --verbose setting
  • tsview: more friendly figsize_img API for the interactive tsview.ipynb in MintPy-tutorial repo
  • plot_network: fix --nodisplay bug while calling from jupyter notebook
  • install: use bash *.sh to replace chmod +x cmd in docs/installation.md and .circleci/config.yml files

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.

+ utils.readfile.read_attribute(): fix a bug of grabbing the UNIT for HDF5 dataset, if `datasetName` is specified with date info as suffix. This only affects the unit display in the colorbar during plot.
+ utils.utils1.get_residual_std/rms(): use np.float32 to replace np.float to avoid numpy deprecation warning
+ view: print WARNING msg regardless of --verbose setting
+ tsview: more friendly figsize_img API for the interactive tsview.ipynb in MintPy-tutorial repo
+ plot_network: fix --nodisplay bug while calling from jupyter notebook
+ install: use bash *.sh to replace chmod +x cmd in docs/installation.md and .circleci/config.yml files
@yunjunz yunjunz merged commit 590ecc0 into insarlab:main Aug 5, 2021
@yunjunz yunjunz deleted the read branch August 5, 2021 18:05
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