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

[WIP] Replace local FDR with Orange's #416

Merged
merged 2 commits into from
May 28, 2019

Conversation

ajdapretnar
Copy link
Collaborator

Issue

Replace local implementation of FDR with core Orange's implementation.

Description of changes

Import FDR from Orange.

Includes
  • Code changes
  • Tests
  • Documentation

@ajdapretnar ajdapretnar changed the title Replace local FDR with Orange's [WIP] Replace local FDR with Orange's Feb 25, 2019
@ajdapretnar
Copy link
Collaborator Author

I'd like to write tests, but have to figure out how to call settings.

@ajdapretnar
Copy link
Collaborator Author

@janezd I'd love to finish this, but I don't know how to test with settings. Would appreciate help. :)

@ajdapretnar
Copy link
Collaborator Author

Travis probably requires the latest version of Orange for tests to pass. I wonder whether we should fix Orange version in here as well.

@codecov-io
Copy link

Codecov Report

Merging #416 into master will increase coverage by 0.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #416      +/-   ##
==========================================
+ Coverage   85.88%   86.06%   +0.18%     
==========================================
  Files          34       34              
  Lines        1963     1931      -32     
  Branches      344      331      -13     
==========================================
- Hits         1686     1662      -24     
+ Misses        236      229       -7     
+ Partials       41       40       -1

@ajdapretnar ajdapretnar merged commit 31aaf29 into biolab:master May 28, 2019
@ajdapretnar ajdapretnar deleted the FRD-from-core branch January 21, 2021 12:56
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