Skip to content

initial prep_NISAR#100

Open
ehavazli wants to merge 9 commits intomainfrom
prep_nisar
Open

initial prep_NISAR#100
ehavazli wants to merge 9 commits intomainfrom
prep_nisar

Conversation

@ehavazli
Copy link
Collaborator

This PR includes a new notebook to prepare NISAR data to run through ATBD methods. Currently only ionospheric correction layer is extracted. A script for extracting layers from NISAR data is obtained from ADT and included here with it's own history. Another PR will follow for the automated extraction and loading of troposphere and solid earth tides layers.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@rzinke
Copy link
Contributor

rzinke commented Oct 16, 2025

Worked for me. Properly set up MintPy/inputs files as expected.

Copy link
Contributor

@katia-tymofyeyeva katia-tymofyeyeva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't merge - the notebook does not work with the rest of the workflow and real NISAR data. Several changes have to be made, including creating the mintpy configuration file, creating a mask file and others. The rest of the workflow also needs some adaptations to work with NISAR data. I am making the changes and will commit as soon as they are ready. Thanks! -Katia

@ehavazli
Copy link
Collaborator Author

ehavazli commented Feb 5, 2026

@katia-tymofyeyeva

  • Can you please be more specific what didn't work for you? I tested again from scratch and everything worked as intended.
  • I added the MintPy configuration file generation.
  • Mask files are generated in later stages from the data (e.g., coherence or connected component based) according to user needs. I don't think it is a good idea to mask the data at this stage.
  • Please specify the other missing files.
  • This PR has been open awaiting review for a long time. The adaptations for the rest of the workflow was not initially part of this PR so I don't think there is a need to discuss them here.

I added my_sites.txt file to this PR so you can test with the sample data and let me know if you find any issues.
Thank you!

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.

4 participants