-
-
Notifications
You must be signed in to change notification settings - Fork 361
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
Add set_equivalence_ratio to SolutionArray objects #677
Add set_equivalence_ratio to SolutionArray objects #677
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had an alternative suggestion for mapping the input equivalence ratio to an array of the appropriate shape.
205a78a
to
201a5f8
Compare
Done and pushed. Let's see whether unit tests do their job ... |
201a5f8
to
c31a351
Compare
Codecov Report
@@ Coverage Diff @@
## master #677 +/- ##
==========================================
+ Coverage 70.62% 70.62% +<.01%
==========================================
Files 372 372
Lines 43565 43565
==========================================
+ Hits 30768 30769 +1
+ Misses 12797 12796 -1
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #677 +/- ##
==========================================
+ Coverage 70.62% 70.62% +<.01%
==========================================
Files 372 372
Lines 43565 43565
==========================================
+ Hits 30768 30769 +1
+ Misses 12797 12796 -1
Continue to review full report at Codecov.
|
unit tests work ;) |
Please fill in the issue number this pull request is fixing:
Fixes #604
Changes proposed in this pull request:
ThermoPhase.set_equivalence_ratio
, and check dimensions