reductionista commented on pull request #514:
URL: https://github.com/apache/madlib/pull/514#issuecomment-682259949


   We should update the usage message (lines 384-394 in 
`madlib_keras_predict.py_in`:
   ```
   The output table ('output_table' above) contains the following columns:      
The output table ('output_table' above) contains the following columns:
   id:                 Gives the 'id' for each prediction, corresponding        
id:                 Gives the 'id' for each prediction, corresponding
                       to each row from the test_table.                     to 
each row from the test_table.
   estimated_COL_NAME: (For pred_type='response') The estimated class for       
estimated_COL_NAME: (For pred_type='response') The estimated class for
                       classification, where COL_NAME is the name of the        
                    classification, where COL_NAME is the name of the
                       column to be predicted from test data.                   
    column to be predicted from test data.
   prob_CLASS:         (For pred_type='prob' for classification) The    
prob_CLASS:         (For pred_type='prob' for classification) The
                       probability of a given class. There will be one          
            probability of a given class. There will be one
                       column for each class in the training data.              
            column for each class in the training data.
                       TODO change this
   ```
   But other than that looks good!


----------------------------------------------------------------
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:
[email protected]


Reply via email to