Skip to content

Restructure documentation and add new sections#525

Merged
rubencalje merged 27 commits intodevfrom
refactor_docs
Dec 11, 2025
Merged

Restructure documentation and add new sections#525
rubencalje merged 27 commits intodevfrom
refactor_docs

Conversation

@rubencalje
Copy link
Collaborator

@rubencalje rubencalje commented Dec 4, 2025

Documentation was reorganized into new sections:

  • data sources
  • utilities
  • workflows
  • advanced stress packages
  • examples

Example notebooks were moved to corresponding subfolders, and new .rst files were added for each section. A new notebook on meteorological data was added. The index.rst was updated to reflect the new structure. Metadata in several notebooks was cleaned up, and a script to clear notebook outputs was introduced.

Documentation was reorganized into new sections: data sources, utilities, workflows, and advanced stress packages. Example notebooks were moved to corresponding subfolders, and new .rst files were added for each section. A new notebook on meteorological data was added. The index.rst was updated to reflect the new structure. Metadata in several notebooks was cleaned up, and a script to clear notebook outputs was introduced.
Expanded the list of available data sources in the documentation for greater clarity. Moved cache_example.py from docs/examples to docs/utilities for better organization.
Revised several documentation notebooks to improve clarity, update code, and remove author credits and images. Added a new workflow notebook for aggregating surface water features into model cells (taken from surface-water-notebook). Updated example and utility notebooks for consistency and improved instructions. Adjusted references to model workspace paths and improved section headings in meteorological data documentation.
Refactored example and workflow notebooks to remove unused script_path arguments from function calls and cleaned up markdown formatting. Updated kernelspec and language_info metadata in relevant notebooks. Minor improvements to documentation text for clarity.
@rubencalje
Copy link
Collaborator Author

I think I have fixed all the errors and added a new logo. This PR is ready for review.

@rubencalje
Copy link
Collaborator Author

I removed all names and dates from the notebooks, as well as the logo of Artesia, as I think it looks better without. In the main page of the documentation, I added the logo of Artesia.

Getting Python
--------------
To install nlmod, a working version of Python 3.7 or higher has to be
To install `nlmod`, a working version of Python 3.7 or higher has to be
Copy link
Collaborator

Choose a reason for hiding this comment

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

Probably good to modify Python 3.7 to something a little more recent.

- Landuse data from BGT, BRT or BRP
- Meteorological data from KNMI or Meteobase
- Waterboard data from each of the 21 waterboards
- Soil phisical units map from BOFEK
Copy link
Collaborator

Choose a reason for hiding this comment

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

physical

Copy link
Collaborator

Choose a reason for hiding this comment

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

Section header levels are a bit weird. I think all below the title should be level 2, i.e. ##

Copy link
Collaborator

Choose a reason for hiding this comment

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

the bullet points under create model header are not correctly rendered.

Copy link
Collaborator

Choose a reason for hiding this comment

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

bullet points under Download data are not rendered correctly.

Copy link
Collaborator

Choose a reason for hiding this comment

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

nlmod.sim.ims can auto register the solution package if the model is passed as well: nlmod.sim.ims(sim, pname="ims_gwt", model=gwt)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Did not change anything, as I am not sure what to do exactly

Copy link
Collaborator

Choose a reason for hiding this comment

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

exuction time typo

Copy link
Collaborator

Choose a reason for hiding this comment

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

structured grid has 3x3 cells not 10x10?

Copy link
Collaborator

Choose a reason for hiding this comment

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

bulleted list not rendering correctly

Copy link
Collaborator

Choose a reason for hiding this comment

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

Some rendering of code in model cells not so pretty?

Copy link
Collaborator

@dbrakenhoff dbrakenhoff left a comment

Choose a reason for hiding this comment

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

I ran through the example notebooks and found some stuff not rendering as intended.

For the rest, a really nice improvement to the documentation!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in NHFLO Dec 9, 2025
Set nb_merge_streams to True in Sphinx configuration to merge output streams during notebook execution.
Added a noop implementation for tqdm and monkey-patched common variants in docs/conf.py to prevent progress bars from appearing during documentation builds.
Introduces a custom.css file to make pandas DataFrames horizontally scrollable in documentation. Updates conf.py to include the new CSS and switches TQDM disabling to use an environment variable for cleaner configuration.
@rubencalje rubencalje merged commit a9f3bb1 into dev Dec 11, 2025
0 of 3 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in NHFLO Dec 11, 2025
@rubencalje rubencalje deleted the refactor_docs branch December 11, 2025 12:58
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