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.

Sign up for GitHub

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

Jump to bottom

Remove the abs() from the score function #634

Merged
merged 2 commits into from Nov 29, 2017
Merged

Remove the abs() from the score function #634

merged 2 commits into from Nov 29, 2017

Conversation

weixuanfu
Copy link
Contributor

What does this PR do?

  1. Make TPOT show raw score instead of absolute score. Some metrics for regression use negative scores.

  2. Fix 2 unit tests due to updated score function.

What are the relevant issues?

#631

Questions:

  • Do the docs need to be updated? No
  • Does this PR add new (Python) dependencies? No

@weixuanfu weixuanfu mentioned this pull request Nov 29, 2017
Closed
@coveralls
Copy link

Coverage Status

Coverage remained the same at 96.65% when pulling 9bc73c6 on weixuanfu:rm_abs into 955f6a2 on rhiever:development.

@rhiever
Copy link
Contributor

rhiever commented Nov 29, 2017

Please go ahead and merge when it passes the unit tests.

@weixuanfu weixuanfu merged commit c127787 into EpistasisLab:development Nov 29, 2017
@weixuanfu weixuanfu deleted the rm_abs branch November 29, 2017 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@weixuanfu @coveralls @rhiever