njayaram2 commented on a change in pull request #430: Add generate mst table
utility.
URL: https://github.com/apache/madlib/pull/430#discussion_r312189427
##########
File path: src/ports/postgres/modules/deep_learning/test/madlib_keras.sql_in
##########
@@ -1285,3 +1285,84 @@ SELECT assert(UPPER(pg_typeof(prob)::TEXT) = 'DOUBLE
PRECISION[]',
'Predict byom failure for null class value and prob pred_type.
Expeceted prob to
be of type DOUBLE PRECISION[]')
FROM iris_predict_byom LIMIT 1;
+
+-- MST talbe generation test
+
+-- Valid inputs should pass and yield 6 msts in the table
+DROP TABLE IF EXISTS mst_table;
+SELECT madlib.madlib_keras_generate_mst_table(
Review comment:
Do not hard-code `madlib` as the schema name. The functions can be called
without the schema name in our dev-checks.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
