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

Updated mass defintions to match pyHalo #48

Merged
merged 3 commits into from
Nov 2, 2023

Conversation

cgannonucm
Copy link
Contributor

Updated parameter files / hdf5 files. Now contain "massHaloEnclosedCurrent" and "concentration" parameters which give definitions for the infall mass and concentration that should match the definitions used in pyHalo.
Additionally, updated TNFW_example.xml to credit author.

@cgannonucm
Copy link
Contributor Author

cgannonucm commented Oct 28, 2023

Now when loading a galacticus file concentrations are used by default to calculate virial radius using the new concentration parameter in the output. This allows the definition of virial radius to match that used in pyHalo.

Additionally, refactor DMFromGalacticus to allow for modifying which parameters get passed when creating TNFWFromParams object. This should allow for more flexible behavior in the future.

@dangilman dangilman merged commit f3d5ede into dangilman:master Nov 2, 2023
1 check 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