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

More user friendly error message #994

Conversation

josiahjohnston
Copy link

Make an error message more user friendly for people who have errors in their custom scoring functions.

What does this PR do?

Give the user another hint as to what may have caused an error.

Where should the reviewer start?

Look at the diff

How should this PR be tested?

Standard tests

Any background context you want to provide?

#983

It would be nice to have an debugging option to make tpot.gp_deap:_wrapped_cross_val_score() print exceptions rather than hiding all exception. It would probably be best to control that via by the verbosity setting, but a separate hacky flag like tpot.gp_deap.display_exceptions=True would be adequate for local debugging purposes.

What are the relevant issues?

#983

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 remained the same at 96.747% when pulling cf26495 on josiahjohnston:user-friendly-error-message into fc26b71 on EpistasisLab:development.

@weixuanfu
Copy link
Contributor

Thank you for the PR and fix. I will merge it to dev branch.

@weixuanfu weixuanfu merged commit 410d88c into EpistasisLab:development Jan 8, 2020
@josiahjohnston josiahjohnston deleted the user-friendly-error-message branch January 8, 2020 20:38
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.

None yet

3 participants