[MADLIB-1316] Simplify madlib_keras_predict interface

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: v1.16
• Component/s: Deep Learning
• Assignee: Unassigned
• Reporter: Nandish Jayaram
• Created: 03/Apr/19 19:26
• Updated: 23/Apr/19 17:27
• Resolved: 23/Apr/19 17:27

Description

The interface for `madlib_keras_predict()` is currently more complex than necessary. The following params that are currently needed can be inferred from model/model_summary table:
1. `model_arch_table`
2. `model_arch_id`
3. `compile_params`

These params are passed to `madlib_keras_fit()`, and we should just store them in the model output summary table. Any model we learn in fit is very specific to these params and they should not be changed while calling predict anyway.

The following proposed predict interface is simpler and sufficient:

madlib_keras_predict(
    model_table            VARCHAR,
    test_table                 VARCHAR,
    id_col                        VARCHAR,
    independent_varname     VARCHAR,
    output_table            VARCHAR
    pred_type                 VARCHAR
)