-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[ENH] LinePlot: data info displayed in the status bar #4633
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4633 +/- ##
==========================================
- Coverage 83.65% 83.62% -0.03%
==========================================
Files 281 276 -5
Lines 56269 55474 -795
==========================================
- Hits 47072 46391 -681
+ Misses 9197 9083 -114 |
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.
The Subset dataset is not necessary to produce the visualisation, therefore I'd exclude it from the input summary.
The same goes for all visualisation widgets.
@@ -543,7 +545,10 @@ def set_data(self, data): | |||
self.group_view.setEnabled(len(self.group_vars) > 1) | |||
self.group_var = self.data.domain.class_var \ | |||
if self.data.domain.has_discrete_class else None | |||
|
|||
self.info.set_input_summary(len(self.data), | |||
format_summary_details(self.data)) |
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'd rather show the info before the data is changed check_data()
.
If one, for instance, inputs the zoo
dataset, no information is shown.
no_data_info = "No data on input." | ||
self.assertEqual(self.widget.infoLabel.text(), no_data_info) | ||
no_data = "No data on input" | ||
self.assertEqual(self.widget.info._StateInfo__input_summary.details, no_data) |
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.
Please keep the line length up to 79 characters.
Description of changes
Input/output data info displayed in the status bar.
Includes