Skip to content

[ADDITION] Periodic boundary conditions#275

Draft
albert-de-montserrat wants to merge 17 commits into
mainfrom
adm/periodic
Draft

[ADDITION] Periodic boundary conditions#275
albert-de-montserrat wants to merge 17 commits into
mainfrom
adm/periodic

Conversation

@albert-de-montserrat

@albert-de-montserrat albert-de-montserrat commented Apr 10, 2026

Copy link
Copy Markdown
Member

Whats the purpose of this PR?

  • Bug fix
  • New feature
  • Documentation update
  • Other, please explain

Describe it in more detail below:

Implements periodic boundary conditions

donut_advection_periodic_dt_0 018

Checklist

  • The PR title is descriptive and starts with the appropriate tag: [BUGFIX], [ADDITION], [DOC], etc.
  • New tests (either assessing the correct behaviour of new internal functions or the correctness of a miniapps or benchmarks) were added, or old tests were updated
  • Affected miniapps have also been updated
  • The new feature was added in a way that does not break public API
  • New documentation related to the new feature was added
  • The new code follows the contributor guidelines, in particular the Runic Style

@codecov

codecov Bot commented Apr 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant