Skip to content

Sentinel-3 Datasets#579

Merged
nanaboamah89 merged 21 commits intomainfrom
sentinel-3-ds
Sep 26, 2025
Merged

Sentinel-3 Datasets#579
nanaboamah89 merged 21 commits intomainfrom
sentinel-3-ds

Conversation

@mickwelli
Copy link
Contributor

@mickwelli mickwelli commented Mar 27, 2025

Proposed changes

Sentinel-3 datasets notebook

Checklist (replace [ ] with [x] to check off)

  • Remove any unused Python packages from Load packages
  • Remove any unused/empty code cells
  • Remove any guidance cells (e.g. General advice)
  • Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended)
  • Include relevant tags in the first notebook cell and re-use tags if possible
  • Ensure appropriate colour schemes have been used to maximise accessibility for vision impairment. Test your images or learn more with Coblis or TPGI
  • Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated

@nanaboamah89 nanaboamah89 self-requested a review August 5, 2025 09:29
@nanaboamah89 nanaboamah89 marked this pull request as ready for review September 9, 2025 14:22
@nanaboamah89
Copy link
Contributor

@mickwelli Kindly review the sentinel 3 loading notebook.

@mickwelli
Copy link
Contributor Author

Hi @nanaboamah89 thanks for the documentation and work on the notebook. I've made the following changes:

  1. Renamed this notebook 'Sentinel-3 Land' because I think we'll have other notebooks that deal with the SYN and Water products.
  2. Added a data masking step.
  3. Removed dataMask and LQSF plots.
  4. Added some text related to the scale and offset.

Let me know what you think of these changes. Thanks.

Copy link
Contributor

@nanaboamah89 nanaboamah89 left a comment

Choose a reason for hiding this comment

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

@mickwelli I reviewed the changes, and everything looks good on my end. Thanks!

Copy link
Contributor

@lisarebelo lisarebelo left a comment

Choose a reason for hiding this comment

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

This dataset notebook is well presented and structured with clear examples. Please check the extra S3 in "Loading Sentinel 3 Sentinel-3 OLCI Data"

@nanaboamah89 nanaboamah89 merged commit 5566944 into main Sep 26, 2025
1 check passed
@nanaboamah89 nanaboamah89 deleted the sentinel-3-ds branch September 30, 2025 07:29
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.

3 participants