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

Feature Constructor: Add some builtin functions #2611

Merged
merged 1 commit into from
Sep 27, 2017

Conversation

janezd
Copy link
Contributor

@janezd janezd commented Sep 22, 2017

Issue

Feature Constructor widget does not offer abs in the combo.

Description of changes

I added abs and, while at it, also some other builtin function that could be useful.

Includes
  • Code changes
  • Tests

@codecov-io
Copy link

codecov-io commented Sep 22, 2017

Codecov Report

Merging #2611 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2611      +/-   ##
==========================================
+ Coverage   74.99%   74.99%   +<.01%     
==========================================
  Files         331      331              
  Lines       58076    58081       +5     
==========================================
+ Hits        43555    43560       +5     
  Misses      14521    14521

@astaric
Copy link
Member

astaric commented Sep 22, 2017

Could you also provide some tests?

@janezd
Copy link
Contributor Author

janezd commented Sep 22, 2017

I was like -- test what? Then I tested whether abs is there --- and it wasn't! :)

Fixed and done.

@jerneju jerneju merged commit c1fa070 into biolab:master Sep 27, 2017
@janezd janezd deleted the feature-constructor-builtins branch April 5, 2019 17:31
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.

4 participants