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.

Sign up for GitHub

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

Jump to bottom

Instead of BSE_Settings let Population use ini files too #37

Closed
TomWagg opened this issue Oct 28, 2022 · 0 comments · Fixed by #107
Closed

Instead of BSE_Settings let Population use ini files too #37

TomWagg opened this issue Oct 28, 2022 · 0 comments · Fixed by #107
Assignees
@TomWagg @katiebreivik
Labels
backburner leave it for now COSMIC An issue related to COSMIC enhancement New feature or request

Comments

@TomWagg
Copy link
Owner

TomWagg commented Oct 28, 2022

As @katiebreivik suggested a while ago (and I am now remembering), it would be nice if Population could use ini files (since that's all the rage in COSMIC these days apparently 🙃).

I'm not entirely sure how this would work since I've not used the files before but I guess we would need to change __init__ and sample_initial_binaries to account for it.
@TomWagg TomWagg added enhancement New feature or request COSMIC An issue related to COSMIC backburner leave it for now labels Oct 28, 2022
@TomWagg TomWagg changed the title Instead of BSE_Settings let Population use ini files too/instead Instead of BSE_Settings let Population use ini files too Oct 11, 2023
@TomWagg TomWagg mentioned this issue Apr 24, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TomWagg TomWagg

@katiebreivik katiebreivik

Labels
backburner leave it for now COSMIC An issue related to COSMIC enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow the use of ini files to specify BSE_settings TomWagg/cogsworth
2 participants
@TomWagg @katiebreivik