Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

switch Strang integration to work with (rho X, rho e)

Open
  • Is something wrong?
#1,200 3 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Stale
Tech stack
cpp
Domain
hpc

Research direction

Start by locating the Strang and SDC integration implementations and compare how each represents (X, e) versus (rho X, rho e). Update the Strang path so both integrations use the same variables, then verify the resulting atol behavior and existing integration tests.

Written by the indexing model from the issue text.

Description

integrator

Right now Strang integrates (X, e) and SDC integrates (rho X, rho e). We should sync them.

algebraically, since rho is constant in Strang integration, this should not affect things much (except how atol is interpreted). But it will make code reuse between SDC and Strang much easier.

Dominant language
C++
Stars
44
Forks
46
Avg merge
3d 7h
Merged PRs (30d)
10

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from AMReX-Astro/Microphysics

All issues in AMReX-Astro/Microphysics

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.