Skip to content

Enable processing of cosicorr pixel offset data#626

Merged
yunjunz merged 12 commits intoinsarlab:mainfrom
forrestfwilliams:pixel_offset
Aug 4, 2021
Merged

Enable processing of cosicorr pixel offset data#626
yunjunz merged 12 commits intoinsarlab:mainfrom
forrestfwilliams:pixel_offset

Conversation

@forrestfwilliams
Copy link
Collaborator

@forrestfwilliams forrestfwilliams commented Aug 3, 2021

Description of proposed changes
References Issue #625
Adds functionality that allows for the loading of cosicorr pixel offset data.

  • adds the prep_cosicorr.py file to generate .rsc files
  • makes small changes to load_data.py and readfile.read to allow for using cosicorr as a processor
  • in ifgram_inversion, disables conversion from pixel to meter for cosicorr offset data, since it is in meters natively

While prep_cosicorr.py is designed for cosicorr files, it will take any geotiff as input, as long as it is accompanied by a metadata file of the form:

fname1.tif date1 date2
fname2.tif date1 date2
...

Reminders

  • 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.
  • If adding new functionality, add a detailed description to the documentation and/or an example.

- load_data: add cosicorr as a processor
- readfile: add cosicorr
- ifgram_inversion: disable conversion from
  pixel to meter for cosicorr, since it is
  in meters natively
@forrestfwilliams
Copy link
Collaborator Author

forrestfwilliams commented Aug 3, 2021

@yunjunz I've put some example data with a cfg file in the test_hyp3 folder you shared with me earlier!

@yunjunz yunjunz self-requested a review August 3, 2021 22:10
@yunjunz
Copy link
Member

yunjunz commented Aug 3, 2021

Thank you @forrestfwilliams, this is awesome! I only have the minor comments above. This PR is good to go in my opinion.

And thank you for sending me an example test data, that is very helpful.

@yunjunz yunjunz merged commit 0e8d7cb into insarlab:main Aug 4, 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.

2 participants