Skip to content

[ENH] OWDiscretize: data info displayed in status bar#4331

Merged
VesnaT merged 1 commit into
biolab:masterfrom
aturanjanin:owdiscretize
Jan 17, 2020
Merged

[ENH] OWDiscretize: data info displayed in status bar#4331
VesnaT merged 1 commit into
biolab:masterfrom
aturanjanin:owdiscretize

Conversation

@aturanjanin
Copy link
Copy Markdown
Contributor

Description of changes

Display input and output data info in the status bar.

Includes
  • Code changes
  • Tests
  • Documentation

@aturanjanin aturanjanin changed the title OWDiscretize: data info displayed in status bar [ENH] OWDiscretize: data info displayed in status bar Jan 14, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 14, 2020

Codecov Report

Merging #4331 into master will increase coverage by 0.22%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #4331      +/-   ##
=========================================
+ Coverage   86.68%   86.9%   +0.22%     
=========================================
  Files         396     396              
  Lines       71510   71851     +341     
=========================================
+ Hits        61988   62442     +454     
+ Misses       9522    9409     -113

Comment thread Orange/widgets/data/owdiscretize.py Outdated

self._update_spin_positions()

self.info.set_output_summary(self.info.NoInput)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When setting output summary, output default message (self.info.NoOutput) should be used.

Comment thread Orange/widgets/data/owdiscretize.py Outdated
output = self.data.transform(domain)
self.info.set_output_summary(len(output))
else:
self.info.set_output_summary(self.info.NoInput)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above.

@VesnaT VesnaT merged commit 2bcef21 into biolab:master Jan 17, 2020
@aturanjanin aturanjanin deleted the owdiscretize branch February 28, 2020 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants