Skip to content

NISAR_DualPol_CloudePottier_DPID notebook added - #36

Open
ials wants to merge 3 commits into
ASFOpenSARlab:mainfrom
ials:feature/sar-polarization-notebook
Open

NISAR_DualPol_CloudePottier_DPID notebook added#36
ials wants to merge 3 commits into
ASFOpenSARlab:mainfrom
ials:feature/sar-polarization-notebook

Conversation

@ials

@ials ials commented Jul 6, 2026

Copy link
Copy Markdown

Illustration of SAR dual polarization concepts, along with two supporting Python utility modules.

Changes Made

  • notebooks/NISAR_DualPol_CloudePottier_DPID.ipynb: Interactive tutorial explaining dual polarization matrices and decomposition.
  • utils/nisar_polsar.py: Python script with SAR processing utilities for NISAR dual-pol GSLC analysis.
  • utils/nisar_plot_utils.py: Python script containing plotting functions for polarization bands.

How to Test

  1. Open notebooks/NISAR_DualPol_CloudePottier_DPID.ipynb in Jupyter.
  2. Run all cells sequentially to verify the plots render correctly using the new utility scripts.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: c637e82
✅ Deployment Preview URL: https://ASFOpenSARlab.github.io/NISAR_Cookbook/_preview/36

@Alex-Lewandowski

Copy link
Copy Markdown
Contributor

@ials Thank you very much for contributing your polarimetry workflow! I have scheduled some time to test and review it on Wednesday.

@review-notebook-app

review-notebook-app Bot commented Jul 28, 2026

Copy link
Copy Markdown

View / edit / reply to this conversation on ReviewNB

Alex-Lewandowski commented on 2026-07-28T17:27:09Z
----------------------------------------------------------------

  • Make the title a level-1 header (using a single # instead of ##)
  • Make the Overview a level-2 header (##)
  • The Overview should be a list of notebook section links. The current overview content could move above the overview section.
  • The template notebook demonstrates how to create linkable notebook sections for the overview
  • Add horizontal rules to separate each notebook section (<hr>)
  • Add a Prerequisite section (see notebook template)
  • Move the author and references to the last markdown cell in the notebook with the title "Resources and references"
  • See the NISAR Cookbook notebook template
  • Remove standalone installation instructions and move Python dependencies into the Cookbook's Pixi environment
  • It worked to add:
  • These to the "feature.isce3.dependencies" section of the pixi.toml:
               hdf5plugin = "*"
               rasterio = "*"
               pyproj = "*"
               scipy = "*"
  • This as a new "feature.isce3.pypi-dependencies" section:
                       [feature.isce3.pypi-dependencies]
                       opencv-python = "*" 

@Alex-Lewandowski

Copy link
Copy Markdown
Contributor

@ials, I apologize that it took a bit longer than intended to start reviewing your notebook. The first comment addresses notebook formatting and handling of the software environment. I will likely have more feedback as I begin to dive into the code. Thank you again for the contribution!

@review-notebook-app

review-notebook-app Bot commented Jul 28, 2026

Copy link
Copy Markdown

View / edit / reply to this conversation on ReviewNB

Alex-Lewandowski commented on 2026-07-28T17:37:18Z
----------------------------------------------------------------

It would be helpful to include some code that searches and downloads a GSLC. There is an example that you could copy/paste from in section 2 of this notebook.


@ials

ials commented Jul 30, 2026

Copy link
Copy Markdown
Author

@Alex-Lewandowski, thank you for your feedback. I've done the changes.

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