Skip to content

[FIX] Preprocess: enums containing function names changed#2151

Merged
lanzagar merged 2 commits intobiolab:masterfrom
jerneju:type-predictions
Mar 31, 2017
Merged

[FIX] Preprocess: enums containing function names changed#2151
lanzagar merged 2 commits intobiolab:masterfrom
jerneju:type-predictions

Conversation

@jerneju
Copy link
Contributor

@jerneju jerneju commented Mar 29, 2017

Issue

When loaded schema from file, function names are loaded as integers. That is why functions cannot be called anymore (int is not callable).
https://sentry.io/biolab/orange3/issues/227119437/

Description of changes

Enums in Scale preprocess now saved as string and not as int anymore.

Includes
  • Code changes
  • Tests
  • Documentation

@codecov-io
Copy link

codecov-io commented Mar 29, 2017

Codecov Report

Merging #2151 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #2151   +/-   ##
=======================================
  Coverage   71.55%   71.55%           
=======================================
  Files         318      318           
  Lines       54470    54470           
=======================================
  Hits        38976    38976           
  Misses      15494    15494

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 412e409...4fb1caa. Read the comment docs.

@lanzagar lanzagar merged commit 9dd66d7 into biolab:master Mar 31, 2017
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