[GitHub] [madlib] fmcquillan99 commented on pull request #518: DL: [AutoML] Add support for Hyperopt on top of MOP for MPP + AutoML best-so-far

[GitBox <gi...@apache.org>]

fmcquillan99 commented on pull request #518:
URL: https://github.com/apache/madlib/pull/518#issuecomment-698536437


   These 2 methods seems to want `hyperopt` module installed even if I don't call it.  It should only error out if I select to use `hyperopt` as an autoML method.
   ```
   %%sql
   DROP TABLE IF EXISTS hb_schedule;
   SELECT madlib.hyperband_schedule ('hb_schedule', 
                                      6,
                                      3,
                                      0);
   SELECT * FROM hb_schedule ORDER BY s DESC, i;
   ```
   produces
   ```
   InternalError: (psycopg2.errors.InternalError_) ImportError: No module named hyperopt (plpython.c:5038)
   CONTEXT:  Traceback (most recent call last):
     PL/Python function "hyperband_schedule", line 7, in <module>
       from deep_learning import madlib_keras_automl
     PL/Python function "hyperband_schedule", line 21, in <module>
   PL/Python function "hyperband_schedule"
   
   [SQL: SELECT madlib.hyperband_schedule ('hb_schedule', 
                                      6,
                                      3,
                                      0);]
   ```
   
   Same happens with `madlib_keras_automl` when I select `hyperband` as the method I want


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org