Skip to content
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] Feature Statistics: Move from prototypes to core #3303

Merged
merged 7 commits into from
Nov 9, 2018

Conversation

pavlin-policar
Copy link
Collaborator

@pavlin-policar pavlin-policar commented Oct 10, 2018

Issue

biolab/orange3-prototypes#165

Description of changes

Move Feature Statistics widget to core.

Includes
  • Code changes
  • Tests
  • Documentation

@ajdapretnar
Copy link
Contributor

Could you include the documentation as well? Thanks!

@codecov
Copy link

codecov bot commented Oct 22, 2018

Codecov Report

Merging #3303 into master will increase coverage by 0.08%.
The diff coverage is 87.11%.

@@            Coverage Diff             @@
##           master    #3303      +/-   ##
==========================================
+ Coverage   82.25%   82.34%   +0.08%     
==========================================
  Files         351      354       +3     
  Lines       62450    63244     +794     
==========================================
+ Hits        51371    52080     +709     
- Misses      11079    11164      +85

@pavlin-policar pavlin-policar force-pushed the feature-statistics branch 3 times, most recently from 6032752 to 020c67b Compare October 23, 2018 19:32
@lanzagar lanzagar added this to the 3.18 milestone Oct 26, 2018
@ajdapretnar
Copy link
Contributor

Are we waiting to stop testing on Py3.4 to merge this one? I see only tests for Py3.4 are failing. Then we should probably wait for tests to pass on 3.7, no?

@pavlin-policar
Copy link
Collaborator Author

Yes, this doesn't work with Python 3.4, once it's removed from CI, this can be merged.

@lanzagar lanzagar changed the title OWFeatureStatistics: Move from prototypes to core [ENH] Feature Statistics: Move from prototypes to core Nov 7, 2018
@lanzagar
Copy link
Contributor

lanzagar commented Nov 7, 2018

Python 3.4 was removed from travis testing, but tests still fail on python 3.5 with PyQt4.
I guess we need to be fine with dropping support for PyQt4 too, before merging (or make this compatible).
Anyone thinks we should keep supporting it, or are we all fine with requiring PyQt5? @ales-erjavec?

@lanzagar lanzagar merged commit 470b866 into biolab:master Nov 9, 2018
@pavlin-policar pavlin-policar deleted the feature-statistics branch November 9, 2018 13:52
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