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

Update notebooks to use the io_lulc_v2 product instead io_lulc #561

Merged
merged 4 commits into from
Nov 26, 2024

Conversation

vikineema
Copy link
Collaborator

@vikineema vikineema commented Nov 25, 2024

Proposed changes

Update notebooks to use the io_lulc_v2 product instead of the archived io_lulc product

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

Closes issues (optional)

  • Closes Issue #000

@vikineema vikineema changed the title Update notebooks to use the io_lulc_v2 product instead of the io_lulc product Update notebooks to use the io_lulc_v2 product instead io_lulc Nov 25, 2024
@vikineema vikineema requested a review from mickwelli November 25, 2024 17:58
@mickwelli
Copy link
Contributor

Thanks @vikineema, all ok, just checking that Landsat surface temp is meant to be included in this PR?

@vikineema
Copy link
Collaborator Author

Thanks @vikineema, all ok, just checking that Landsat surface temp is meant to be included in this PR?

Yes, there was a fix required for the notebook although it doesn't use the io_lulc product.

@vikineema vikineema merged commit f233391 into main Nov 26, 2024
1 check passed
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