Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(apr) adding APR #595

Merged
merged 29 commits into from
Oct 20, 2024
Merged

(apr) adding APR #595

merged 29 commits into from
Oct 20, 2024

Conversation

becnealon
Copy link
Contributor

Type of PR:
new feature

Description:
Adding adaptive particle refinement as described in Nealon and Price 2024.

Testing:
Basic test present but more to come.

Did you run the bots? yes

Did you update relevant documentation in the docs directory? yes

Copy link
Owner

@danieljprice danieljprice left a comment

Choose a reason for hiding this comment

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

had a look over, all looks good to me, just a few minor things...

src/main/part.F90 Outdated Show resolved Hide resolved
src/main/readwrite_infile.F90 Outdated Show resolved Hide resolved
src/main/readwrite_infile.F90 Outdated Show resolved Hide resolved
src/main/relaxem.f90 Show resolved Hide resolved
src/setup/relax_star.f90 Outdated Show resolved Hide resolved
src/setup/setup_star.f90 Outdated Show resolved Hide resolved
src/setup/setup_star.f90 Outdated Show resolved Hide resolved
src/setup/setup_wave.f90 Outdated Show resolved Hide resolved
src/setup/setup_wave.f90 Outdated Show resolved Hide resolved
src/tests/test_kdtree.F90 Show resolved Hide resolved
@danieljprice danieljprice merged commit cdb7a8c into danieljprice:master Oct 20, 2024
180 checks passed
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