-
Notifications
You must be signed in to change notification settings - Fork 12
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 costs #80
add costs #80
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #80 +/- ##
===========================================
- Coverage 60.03% 59.08% -0.95%
===========================================
Files 5 5
Lines 603 633 +30
Branches 128 137 +9
===========================================
+ Hits 362 374 +12
- Misses 217 231 +14
- Partials 24 28 +4
Continue to review full report at Codecov.
|
petab/core.py
Outdated
|
||
# read noise distributions from measurement file | ||
|
||
grouping_cols = [col for col in |
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.
Need to check for all-NaN, otherwise grouping will fail
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.
use core.get_notnull_columns
petab/lint.py
Outdated
@@ -220,6 +219,7 @@ def measurement_table_has_timepoint_specific_mappings(measurement_df): | |||
grouped_df2 = grouped_df.groupby(grouping_cols).size().reset_index() | |||
|
|||
if len(grouped_df.index) != len(grouped_df2.index): | |||
logger.warn(grouped_df) |
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.
This should come with a message, not?
Check whether there are not multiple noise distributions for an | ||
observable, and that the names are correct. | ||
""" | ||
df = measurement_df.copy() |
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.
Should also check for valid values, not only for uniqueness?
No description provided.