-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fix combinations of concentration ranges for RMG SA #131
base: main
Are you sure you want to change the base?
Conversation
d163293
to
8cc2d23
Compare
temperature=(500.0, 'K'), | ||
pressure=(1.0, 'bar'), | ||
initialMoleFractions={ | ||
'FA': 4.0, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
Dictionary key 'FA' is subsequently
overwritten
initialMoleFractions={ | ||
'FA': 4.0, | ||
'N2': 3.76, | ||
'FA': 2.5, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
temperature=(500.0, 'K'), | ||
pressure=(10.0, 'bar'), | ||
initialMoleFractions={ | ||
'FA': 4.0, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
Dictionary key 'FA' is subsequently
overwritten
initialMoleFractions={ | ||
'FA': 4.0, | ||
'N2': 3.76, | ||
'FA': 2.5, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
temperature=(500.0, 'K'), | ||
pressure=(100.0, 'bar'), | ||
initialMoleFractions={ | ||
'FA': 4.0, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
Dictionary key 'FA' is subsequently
overwritten
initialMoleFractions={ | ||
'FA': 4.0, | ||
'N2': 3.76, | ||
'FA': 2.5, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
temperature=(2000.0, 'K'), | ||
pressure=(10.0, 'bar'), | ||
initialMoleFractions={ | ||
'FA': 4.0, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
Dictionary key 'FA' is subsequently
overwritten
initialMoleFractions={ | ||
'FA': 4.0, | ||
'N2': 3.76, | ||
'FA': 2.5, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
temperature=(2000.0, 'K'), | ||
pressure=(100.0, 'bar'), | ||
initialMoleFractions={ | ||
'FA': 4.0, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
Dictionary key 'FA' is subsequently
overwritten
initialMoleFractions={ | ||
'FA': 4.0, | ||
'N2': 3.76, | ||
'FA': 2.5, |
Check warning
Code scanning / CodeQL
Duplicate key in dict literal Warning test
overwritten
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #131 +/- ##
==========================================
+ Coverage 73.46% 75.30% +1.83%
==========================================
Files 22 24 +2
Lines 2891 3317 +426
Branches 762 854 +92
==========================================
+ Hits 2124 2498 +374
- Misses 552 586 +34
- Partials 215 233 +18
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
For use case #3 of this method
Fixes the
get_species_concentration_lists_from_ranged_params()
method inRMGConstantTP()
, adding tests for the different use cases.