-
Notifications
You must be signed in to change notification settings - Fork 129
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
S2c updates jag #1301
S2c updates jag #1301
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
core.5902
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this accidently commited? Might need to delete and then commit/push the change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deleted the file and committed again, also changed the base to the correct branch s2c_updates instead of develop
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great @jennaguffogg - feel free to "squash and merge"!
⭐ This is a template you can follow to help construct a good pull request!
Proposed changes
dea-notebooks/Real_world_examples/Turbidity_animated_timeseries.ipynb
dea-notebooks/DEA_products/DEA_Sentinel2_Surface_Reflectance.ipynb
Checklist
(Replace
[ ]
with[x]
to check off)If this is a notebook, then have you:
jupyterlab_code_formatter
tool can be used to format code cells to a consistent style: select each code cell, then clickEdit
and then one of theApply X Formatter
options (YAPF
orBlack
are recommended).Notebook currently compatible with
line below the notebook title to reflect the environments the notebook is compatible with