njayaram2 commented on a change in pull request #370: DL: Support response and
prob prediction outputs
URL: https://github.com/apache/madlib/pull/370#discussion_r276836962
##########
File path: src/ports/postgres/modules/deep_learning/madlib_keras.py_in
##########
@@ -198,11 +199,11 @@ def fit(schema_madlib, source_table, model,
dependent_varname,
final_validation_loss = validation_aggregate_loss[-1]
version = madlib_version(schema_madlib)
class_values, class_values_type = get_col_value_and_type(
- fit_validator.source_summary_table, CLASS_VALUES_COLNAME)
+ fit_validator.source_summary_table, CLASS_VALUES_CNAME)
Review comment:
Variables for some column names were getting very big, hence `CNAME`. I
don't feel strongly between CNAME or COLNAME, so have changed it to COLNAME,
and it's just a couple of chars anyway, so why not! :)
----------------------------------------------------------------
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
