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

The preset satellite component does not provide satellite orbital position/velocity #504

Open
abensonca opened this issue Nov 14, 2023 · 1 comment
Labels
components enhancement New feature or request ram pressure satellites Relating to properties and physics of satellite halos/galaxies

Comments

@abensonca
Copy link
Collaborator

This is problematic as it means that, e.g. hotHaloRamPressureForceOrbitalPosition can not work with this component as it requires the instantaneous position and velocity of a satellite.

A solution would be to make the preset satellite component simply (and, perhaps, optionally) compute position and velocity from the positions of the node and its host. In the typical application of the preset satellite component - N-body merger trees - we likely also have preset positions, so this would be possible.

@abensonca abensonca added enhancement New feature or request satellites Relating to properties and physics of satellite halos/galaxies components ram pressure labels Nov 14, 2023
@abensonca
Copy link
Collaborator Author

To make this most useful, the preset position component should then interpolate positions/velocities between timesteps. This could be done using the nodeOperatorPositionInterpolated class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
components enhancement New feature or request ram pressure satellites Relating to properties and physics of satellite halos/galaxies
Projects
None yet
Development

No branches or pull requests

1 participant