-
Notifications
You must be signed in to change notification settings - Fork 247
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
Image property outliers display when there are no outliers #1738
Image property outliers display when there are no outliers #1738
Conversation
…ay and test the case of black and white
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.
Looks good! Can you just upload a pic of how it looks like after the refactor?
display.append(html) | ||
grouped = no_outliers.groupby(level=0).unique().str.join(', ') | ||
grouped_df = pd.DataFrame(grouped, columns=['Properties']) | ||
grouped_df['Reason'] = grouped_df.index |
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.
@shir22 Any idea for a better name than "Reason"?
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.
Suggestion:
- Rename the "Properties Without Outliers" to - "Properties With No Outliers Found"
- Swap order (properties list in the index/first column and the "why" in the following column)
- Ideas for alternative titles to "Reason" (not sure if they're better): More Info / Outlier Detection Results
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.
@shir22 I think that the "why" should be on the left because it's the grouping rule
grouped_df = pd.DataFrame(grouped, columns=['Properties']) | ||
grouped_df['Reason'] = grouped_df.index | ||
grouped_df = grouped_df[['Reason', 'Properties']] | ||
display.append('<h5><b>Properties Without Outliers</h5></b>') |
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.
maybe missing an if?
if not grouped_df.empty():
display.append('<h5><b>Properties Without Outliers</h5></b>')
display.append(grouped_df.style.hide_index())
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.
@matanper, correct- fixed it
The properties without outliers grouped by the reason for not having outliers.
Headline size for the properties is now smaller than the "additional outputs" title.