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

For issue #236 #243

Merged
merged 1 commit into from
Aug 29, 2016
Merged

Conversation

weixuanfu
Copy link
Contributor

[please review the contribution guidelines prior to submitting your pull request. go ahead and delete this line if you've already reviewed said guidelines.]

What does this PR do?

Make TPOT fail gracefully when all pipelines fail to evaluate (issue #236)

Where should the reviewer start?

From Line 307 in tpot.py

How should this PR be tested?

Run the codes with the data in the issue #233

Any background context you want to provide?

Just add a 'if' sentence to determine whether all pipelines are failed

What are the relevant issues?

The issue #236 links to the issue #233 and issue #234

Screenshots (if appropriate)

Questions:

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

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.09%) to 89.699% when pulling 89c19ac on weixuanfu2016:weixuan-contribution into a381392 on rhiever:master.

@rhiever rhiever merged commit c0ae397 into EpistasisLab:master Aug 29, 2016
@rhiever
Copy link
Contributor

rhiever commented Aug 29, 2016

Thank you @weixuanfu2016!

@weixuanfu weixuanfu deleted the weixuan-contribution branch September 20, 2016 19:25
@weixuanfu weixuanfu restored the weixuan-contribution branch June 15, 2017 17:06
@weixuanfu weixuanfu deleted the weixuan-contribution branch June 15, 2017 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants