-
Notifications
You must be signed in to change notification settings - Fork 3
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
skip empty groups in new sources groupby-apply #632
Conversation
7b7b8f4
to
cfb7fab
Compare
@@ -79,6 +79,9 @@ def get_image_rms_measurements( | |||
The group dataframe with the 'img_diff_true_rms' column added. The | |||
column will contain 'NaN' entires for sources that fail. | |||
""" | |||
if len(group) == 0: |
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.
Does group have a .empty
flag like the dataframes?
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.
Yes, they appear to be Pandas DataFrames within that function. As far as I can tell, len(df) == 0
and df.empty
are equivalent in this case.
cfb7fab
to
ba02eb8
Compare
Fixes #631