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

adjusted error message #1031

Merged
merged 2 commits into from
Mar 16, 2020
Merged

Conversation

jamesjjcondon
Copy link
Contributor

What does this PR do?

Includes example of common error source

Where should the reviewer start?

n/a

How should this PR be tested?

n/a

Any background context you want to provide?

I came across this when first using TPOT with binary classes, binary-only evaluation metrics and nans that were treated as a third class. This is likely a common use-case and error for beginners.
(my first contribution : )

What are the relevant issues?

[you can link directly to issues by entering # then the number of the issue, for example, #3 links to issue 3]

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.757% when pulling d0c590a on jamesjjcondon:jamesjjcondon into 09d7645 on EpistasisLab:development.

@weixuanfu
Copy link
Contributor

Thank you for rewording!

@weixuanfu weixuanfu merged commit 9a2da80 into EpistasisLab:development Mar 16, 2020
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.

3 participants