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
Text featurizer warning on evalml import #1017
Comments
Repro
Root cause
Seems like we should do what we can to avoid raising warnings at import-time. Ideas for a fix
def _all_estimators():
return get_importable_subclasses(Estimator, used_in_automl=False)
...
I suggest we do items 1 and 2. |
When #1022 was merged - the runtime warning was removed. However, I'm still seeing the following warning:
@dsherry I'll put up the PR to do items 1 and 2 as you mentioned above but @eccabay has informed me that this warning is not coming from our side. |
After discussing with @rwedge there seems to be two solutions to the above featuretools warning:
I suggest we wait for the new featuretools release and confirm if the warning is still an issue. |
Thanks @jeremyliweishih ! I agree option 2 here is best for handling the featuretools warning. That aside, my understanding is you're currently working on options 1 and 2 from my comment from last week, correct? I think both of those changes, particularly the first, will add value here. |
nlp-primitives 1.0 is out on pypi now and tensorflow is no longer a required dependency |
Awesome, thanks both! |
In each
user_guide
notebook - RTD produces a warning whereverevalml
is first imported:Example
The cause of this issue is
get_importable_subclasses
initializing every component.The text was updated successfully, but these errors were encountered: