Skip to content

Enabling to pass surface composition for B' calculation - #279

Merged
mgoodson-cvd merged 6 commits into
mutationpp:masterfrom
capriatim:master
Jun 15, 2026
Merged

Enabling to pass surface composition for B' calculation#279
mgoodson-cvd merged 6 commits into
mutationpp:masterfrom
capriatim:master

Conversation

@capriatim

Copy link
Copy Markdown
Collaborator

No description provided.

@rdbisme
rdbisme requested a review from mgoodson-cvd November 22, 2025 16:55

@mgoodson-cvd mgoodson-cvd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I haven't tested this yet, but just from looking at the code, I don't see any reason it shouldn't work.

Comment thread interface/fortran/cwrapper.cpp Outdated
Comment thread src/apps/bprime.cpp Outdated
Comment thread src/thermo/Thermodynamics.cpp Outdated
Comment thread src/thermo/Thermodynamics.h Outdated
capriatim added a commit to capriatim/Mutationpp that referenced this pull request Jun 10, 2026
capriatim added a commit to capriatim/Mutationpp that referenced this pull request Jun 10, 2026
@capriatim

Copy link
Copy Markdown
Collaborator Author

@mgoodson-cvd thank you for the review. I addressed the suggested points in the last push.

@mgoodson-cvd mgoodson-cvd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for addressing my comments. I appreciate the change to preserve backwards compatibility, as well as the version bump. Just a few minor things that I might change, though of course it is fine as-is.

Comment thread src/thermo/Thermodynamics.h Outdated
Comment thread src/thermo/Thermodynamics.cpp Outdated
Comment thread src/thermo/Thermodynamics.cpp Outdated
Comment thread src/thermo/Thermodynamics.cpp Outdated
Comment thread src/thermo/Thermodynamics.cpp Outdated
capriatim added a commit to capriatim/Mutationpp that referenced this pull request Jun 10, 2026
@capriatim

Copy link
Copy Markdown
Collaborator Author

Thank you @mgoodson-cvd . I pushed the last modifications

@mgoodson-cvd

Copy link
Copy Markdown
Collaborator

Looks good to me.

@mgoodson-cvd

Copy link
Copy Markdown
Collaborator

@capriatim Now that all checks are green, I'm going to pull and test, will merge as soon as that is done, hopefully today.

@mgoodson-cvd

Copy link
Copy Markdown
Collaborator

All good, @capriatim rebase on latest master and I'll merge.

@mgoodson-cvd
mgoodson-cvd merged commit 4ac2735 into mutationpp:master Jun 15, 2026
43 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.

2 participants