Skip to content

added thematic to sandbox#567

Merged
nanaboamah89 merged 9 commits intomainfrom
thematic
Jan 21, 2025
Merged

added thematic to sandbox#567
nanaboamah89 merged 9 commits intomainfrom
thematic

Conversation

@nanaboamah89
Copy link
Contributor

@nanaboamah89 nanaboamah89 commented Jan 15, 2025

Adding a new thematic notebook to the main page to organize tools into thematic categories

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

Copy link
Contributor

@mickwelli mickwelli left a comment

Choose a reason for hiding this comment

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

Thanks @nanaboamah89, the formatting is very nice. A few points:

  • Is there a reason why some links still have the .ipynb extension? Good to remove if possible.
  • Similarly, could the underscores _ be removed from titles?
  • Could the use of the capitals be consistent? e.g. 'Crop Health' rather than 'Crop health'.

Will this notebook open on startup like the info notebook?

@nanaboamah89
Copy link
Contributor Author

Thanks @mickwelli , I would work on the feedback above.
I am thinking of adding a landing page that would load on startup, I am working on it and would share it with you for your feedback.

@mickwelli
Copy link
Contributor

Thanks @nanaboamah89, this is a great initiative. I made some edits to fix links and remove file extensions from some titles. A few further points:

  • Could the Machine Learning in the ODC Notebook be added to the ML thematic? So it would appear in both the Land Cover and ML thematic
  • Why does the title for ERA5 Climate include 'AWS'? Minor point but I think we can remove this from the title now as we edited the function to refer to Google.
  • It looks like there are duplicate entries for Water Observations from Space under the Water thematic. The first one refers to a non-existent link in Use Cases.
  • Could some introductory text be added at the top of the page somewhere saying something like "This page organises notebooks by theme. To access notebooks, make a single click on the link." Otherwise I think users will double click and reveal the markdown.

Copy link
Contributor

@mickwelli mickwelli left a comment

Choose a reason for hiding this comment

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

Thanks @nanaboamah89, looks great. Nice advancement!

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.

Thank you Edward for actioning this, it is a great step in making our notebooks more accessible and discoverable.

A couple of minor edits:

  1. Can you change the colour and font of the headings banners to be black (background) and plain white text (same as the banner logo at the top).
  2. On the landing page in each of the 3 sections it says "single click here to access the page" Can you change to "Click to access"

@nanaboamah89 nanaboamah89 merged commit 038f84c into main Jan 21, 2025
1 check passed
@mickwelli mickwelli deleted the thematic branch September 23, 2025 09:16
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