Skip to content

rollback to h5py native fancy indexing with version<3#474

Merged
yunjunz merged 1 commit intoinsarlab:mainfrom
yunjunz:h5py
Nov 15, 2020
Merged

rollback to h5py native fancy indexing with version<3#474
yunjunz merged 1 commit intoinsarlab:mainfrom
yunjunz:h5py

Conversation

@yunjunz
Copy link
Copy Markdown
Member

@yunjunz yunjunz commented Nov 14, 2020

Description of proposed changes

  • docs/conda.txt

    • add h5py<3 constrain to avoid the mix indexing bug from h5py v3+
    • add xarray and zarr for the new pyresample.
  • rollback to h5py native fancy indexing in objects/stack.h5 and utils/readfile.py, following fix bool array index bug for h5 dataset #470.

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.

@yunjunz yunjunz force-pushed the h5py branch 2 times, most recently from dd862a0 to da112ec Compare November 15, 2020 01:08
+ docs/conda:
   - add h5py<3 constrain to avoid the mix indexing bug from h5py v3+
   - add python<3.9 constrain to avoid dask imcompatiability
   - add xarray and zarr for the new pyresample.
   - re-organize with notes for pyaps and pyresample separated from mintpy navtive code

+ update docs/conda_env.yml, docs/requirements.txt and setup.py to be consistent with the latest conda.txt

+ obj/stack.ifgramStack.temporal_average(): auto row_step
   - auto calc for row_step if input value is -1 (default)
   - fix ref_yx out of coverage bug
   - rollback to h5py native fancy indexing
   - check dataset shape before calling np.squeeze()

+ utils/readfile:
   - rollback to h5py native fancy indexing
   - check dataset shape before calling np.squeeze()

+ ts2vel:
   - remove sklearn from conda.txt and add ImportError for sklearn
   - del ts_data right after the last usage to save memory

+ test_sbApp: update for geocoded velocity png file
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