Skip to content

Conversation

@reint-fischer
Copy link
Contributor

@reint-fischer reint-fischer commented Dec 16, 2025

This PR adds a guide describing how to choose an appropriate dt and integration scheme. It is based on @michaeldenes notebook from the 10-years Parcels event.

The basics of how to write a custom kernel are now in the Parcels conceptual workflow explanation, and I have added a bit more text and figures to help users decide what to consider in their simulation setup. @michaeldenes I am mainly interested in your thoughts on how detailed this should be and how to generalize the advice we can give in this guide. Curious to hear what you think!

@reint-fischer reint-fischer marked this pull request as ready for review December 17, 2025 16:55
Copy link
Member

@erikvansebille erikvansebille left a comment

Choose a reason for hiding this comment

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

Great notebook! A few suggestions below

<!-- examples/tutorial_kernelloop.ipynb -->

```{toctree}
:caption: Run a simulation
Copy link
Member

Choose a reason for hiding this comment

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

The other sections start with a How-to guide (with a book emoji), this one doesn't. Is that a problem?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it’s not a problem; explanations (with the book emoji) should only exist for more complicated components that benefit from background reading. For “Create ParticleSets” and “Run a simulation” I think the “Parcels conceptual workflow” is a sufficient explanation.

@reint-fischer reint-fischer merged commit d4bbfc2 into v4-dev Dec 19, 2025
11 checks passed
@reint-fischer reint-fischer deleted the how-to-accuracy branch December 19, 2025 11:51
@github-project-automation github-project-automation bot moved this from Backlog to Done in Parcels development Dec 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants