Closed janezd closed 2 years ago
Merging #210 (d917c20) into master (238da17) will increase coverage by
0.34%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #210 +/- ##
==========================================
+ Coverage 69.81% 70.16% +0.34%
==========================================
Files 7 7
Lines 858 858
Branches 163 163
==========================================
+ Hits 599 602 +3
+ Misses 210 209 -1
+ Partials 49 47 -2
Impacted Files | Coverage Δ | |
---|---|---|
orangecontrib/timeseries/timeseries.py | 87.64% <100.00%> (+1.68%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 238da17...d917c20. Read the comment docs.
Issue
Fixes #106.
Description of changes
Problem when the user selected an attribute was caused by a check whether the column contains any nans.
I cannot replicate the bug when using a sequence. Maybe this was already fixed somewhere. @ajdapretnar, please check.
The second commit updates the widget's code to use
QGridLayout
, for more accurate spacingWidgetPreview
Includes