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

Do not filter training data in model.py but on component side #6930

Closed
wants to merge 2 commits into from

Conversation

tabergma
Copy link
Contributor

@tabergma tabergma commented Oct 6, 2020

Proposed changes:
closes #6919

Status (please check what you already did):

  • added some tests for the functionality
  • updated the documentation
  • updated the changelog (please check changelog for instructions)
  • reformat files using black (please check Readme for instructions)

@tabergma
Copy link
Contributor Author

tabergma commented Oct 6, 2020

will create a new PR that will go into the 2.0 release

@tabergma tabergma closed this Oct 6, 2020
@tabergma tabergma deleted the fix-6919 branch October 9, 2020 12:42
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.

sklearn.exceptions.NotFittedError: Vocabulary not fitted or provided
1 participant