Skip to content
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

Up to date - update #31

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

anurag-kulshrestha
Copy link
Collaborator

#30
Dear Gert and Freek,

First of all, thanks for creating this library. During the course of my PhD at ITC, I have used this repository for generating Sentinel-1 interferograms.

  • Over the course of last 6 years, a lot of dependencies have changed, for instance: URL to SRTM DEMs and URLs to Sentinel-1 orbit data. I have tried to make this repository up-to-date as on this post date.

  • There have been several other changes, some of which have been experimental, e.g. def.oversample() for testing oversampling of S1 data, some have been to add parentheses to the print command.

  • A major correction has been made in the geoidal undulation correction step in the create_dem() function. Also, the correct H2PH variable has been added in the input card generation template.

  • Minor changes w.r.t. import statements, compiler choices, etc. have also been done.

  • There might have been some updates with the same statements. Please ignore them. Please also ignore the changes made in the referencephase.cc file.

Could you please review my changes and add them to the main branch as you deem fit?

Kind regards,
Anurag Kulshrestha

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