Skip to content

[FIX] Rank: Fix error with Random forest#4457

Merged
janezd merged 1 commit intobiolab:masterfrom
PrimozGodec:rank-fix
Feb 26, 2020
Merged

[FIX] Rank: Fix error with Random forest#4457
janezd merged 1 commit intobiolab:masterfrom
PrimozGodec:rank-fix

Conversation

@PrimozGodec
Copy link
Copy Markdown
Contributor

Issue

Fixes #4437

Description of changes

When no class variable random forest fails with TypeError since it cannot decides either on classifier or regressor. It is now caught in the widget.

Includes
  • Code changes
  • Tests
  • Documentation

@PrimozGodec PrimozGodec changed the title Rank: fix error with scorers that can be either regressor or classifier. Rank: Fix error with Random forest Feb 26, 2020
@PrimozGodec PrimozGodec changed the title Rank: Fix error with Random forest [FIX] Rank: Fix error with Random forest Feb 26, 2020
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 26, 2020

Codecov Report

Merging #4457 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4457      +/-   ##
==========================================
- Coverage   83.13%   83.13%   -0.01%     
==========================================
  Files         268      268              
  Lines       53914    53914              
==========================================
- Hits        44821    44819       -2     
- Misses       9093     9095       +2

@janezd janezd merged commit 14c3206 into biolab:master Feb 26, 2020
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.

Rank crashes when getting data without target

2 participants