aws / sagemaker-scikit-learn-extension

A library of additional estimators and SageMaker tools based on scikit-learn
Apache License 2.0
39 stars 33 forks source link

Fix _VectorizerMixin import #46

Open zachmayer opened 11 months ago

zachmayer commented 11 months ago

I dunno why from sklearn.feature_extraction.text import VectorizerMixin doesn't work anymore, but from sklearn.feature_extraction.text import _VectorizerMixin does.

Issue #, if available: https://github.com/aws/sagemaker-scikit-learn-extension/issues/42

Description of changes: Add a _

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

zachmayer commented 8 months ago

how do I get these tests to pass and the PR reviewed and merged? is this project still alive?