Skip to content

[FIX] Feature constructor did not restore features when loading from saved workflow#2996

Merged
lanzagar merged 2 commits intobiolab:masterfrom
astaric:fix-feature-constructor-settings-handler
May 11, 2018
Merged

[FIX] Feature constructor did not restore features when loading from saved workflow#2996
lanzagar merged 2 commits intobiolab:masterfrom
astaric:fix-feature-constructor-settings-handler

Conversation

@astaric
Copy link
Copy Markdown
Member

@astaric astaric commented Apr 13, 2018

Issue

When feature expression in feature constructor widget contained anything else than simple variable names, they were not restored when opening a saved workflow.

Description of changes

Update matching logic to support variable names containing special characters and usage of builtin functions.

Includes
  • Code changes
  • Tests
  • Documentation

Update context matching to correctly match expressions that contain
builtin functions and/or variables with comlpex names.
@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 13, 2018

Codecov Report

Merging #2996 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2996      +/-   ##
==========================================
+ Coverage   81.88%   81.89%   +0.01%     
==========================================
  Files         329      329              
  Lines       56867    56879      +12     
==========================================
+ Hits        46564    46583      +19     
+ Misses      10303    10296       -7

@lanzagar lanzagar changed the title [FIX] Feature constructor does not restore features when loading from saved workflow [FIX] Feature constructor did not restore features when loading from saved workflow May 11, 2018
@lanzagar lanzagar merged commit 551e531 into biolab:master May 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet