-
-
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] Data Info display data set name #3187
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3187 +/- ##
==========================================
+ Coverage 82.06% 82.13% +0.06%
==========================================
Files 351 351
Lines 61898 62168 +270
==========================================
+ Hits 50799 51061 +262
- Misses 11099 11107 +8 |
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.
Name
should also be added to the report.
Also, I noticed this widget still uses "discrete" in several places. This should be changed to "categorical". Maybe it can be added to the same PR?
Orange/widgets/data/owdatainfo.py
Outdated
if hasattr(data, "name"): | ||
self.data_set_name = data.name | ||
else: | ||
self.data_set_name = "N/A" |
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.
self.data_set_name = getattr(data, "name", "N/A")
3512bad
to
41c5f3b
Compare
Issue
No data set name displayed in the info box. This could be important when checking if the names of the data sets are set correctly.
Description of changes
Includes the box with the data set name.
Includes