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

Update inject.f90 #554

Merged
merged 2 commits into from
Jun 3, 2024
Merged

Update inject.f90 #554

merged 2 commits into from
Jun 3, 2024

Conversation

crislong
Copy link
Contributor

@crislong crislong commented Jun 3, 2024

Type of PR:
Modification of existing code

Description:
When injecting particles at rinj with Keplerian velocity, we do not take into account the velocity and position of the central object

Testing:
no tests

Did you run the bots? no

Did you update relevant documentation in the docs directory? no

When injecting particles at rinj with Keplerian velocity, we do not take into account the velocity and position of the central object
- flag to follow the sink
- adding 2 simmetric particles to better conserve momentum and avoid shift of com
@danieljprice danieljprice merged commit 411f1d7 into danieljprice:master Jun 3, 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.

None yet

2 participants