Skip to content

[ENH] Correlations: Move from prototypes to core#3362

Merged
lanzagar merged 2 commits intobiolab:masterfrom
VesnaT:correlations
Nov 12, 2018
Merged

[ENH] Correlations: Move from prototypes to core#3362
lanzagar merged 2 commits intobiolab:masterfrom
VesnaT:correlations

Conversation

@VesnaT
Copy link
Contributor

@VesnaT VesnaT commented Nov 7, 2018

Issue
Description of changes
Includes
  • Code changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Nov 7, 2018

Codecov Report

Merging #3362 into master will increase coverage by 0.04%.
The diff coverage is 90.81%.

@@            Coverage Diff            @@
##           master   #3362      +/-   ##
=========================================
+ Coverage   82.26%   82.3%   +0.04%     
=========================================
  Files         351     353       +2     
  Lines       62450   62755     +305     
=========================================
+ Hits        51372   51652     +280     
- Misses      11078   11103      +25

@VesnaT VesnaT force-pushed the correlations branch 3 times, most recently from d1daaf8 to 898ea2d Compare November 7, 2018 11:22
@lanzagar lanzagar added this to the 3.18 milestone Nov 7, 2018
@VesnaT VesnaT force-pushed the correlations branch 2 times, most recently from 39e16ae to b566591 Compare November 9, 2018 10:13
@lanzagar
Copy link
Contributor

lanzagar commented Nov 9, 2018

Add Correlations widget (without input) -> press Start -> crash

@PrimozGodec
Copy link
Contributor

PrimozGodec commented Nov 9, 2018

I think the button should be disabled when no input. It is disabled after changing the correlation but not when the widget is placed at the canvas.

@lanzagar lanzagar changed the title OWCorrelations: Move from prototypes to core [ENH] Correlations: Move from prototypes to core Nov 9, 2018
@lanzagar lanzagar merged commit aea66c6 into biolab:master Nov 12, 2018
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.

3 participants