Skip to content

Conversation

@jeffriley
Copy link
Collaborator

(a) fixed typo in Options.cpp for option --common-envelope-allow-immediate-RLOF-post-CE-survive (was typed common-envelope-allow-immediate-RLOF-post-CE_survive)
(b) fixed typo in Options.cpp for option --common-envelope-allow-radiative-envelope-survive (was typed common-envelope-allow-radiative-envelope-surive)

(neither of these caused problems because Boost matches only as many characters as necessary to determine the option name - would have if the names were not unique up to the typos)

Updated all pythonSubmit files - all are tested functioning.

Also restored examples/methods_paper_plots/detailed_evolution/gw151226evol.png (which I somehow managed to remove with the last PR)

(a) fixed typo in Options.cpp for option --common-envelope-allow-immediate-RLOF-post-CE-survive (was typed common-envelope-allow-immediate-RLOF-post-CE_survive)
(b) fixed typo in Options.cpp for option --common-envelope-allow-radiative-envelope-survive (was typed common-envelope-allow-radiative-envelope-surive)
    (neither of these caused problems because Boost matches only as many characters as necessary to determine the option name - would have if the names were not unique up to the typos)
@jeffriley jeffriley requested a review from ilyamandel November 1, 2021 07:36
@jeffriley jeffriley self-assigned this Nov 1, 2021
Copy link
Collaborator

@ilyamandel ilyamandel left a comment

Choose a reason for hiding this comment

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

Thank you for catching my bugs, @jeffriley -- very embarrassing! :)

@ilyamandel ilyamandel merged commit ff0e0c6 into dev Nov 1, 2021
@ilyamandel ilyamandel deleted the more-minor-fixes branch November 1, 2021 08:20
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.

3 participants