[GitHub] incubator-madlib-site issue #8: MLP demo

[fmcquillan99]

Github user fmcquillan99 commented on the issue:

    https://github.com/apache/incubator-madlib-site/pull/8
  
    This is a really great demo workbook, thanks for posting.
    
    Suggestions
    
    1) concat does not work for greenplum, perhaps do:
    
    SELECT round(count(*)*100/60000.0,2) as train_accuracy_percent from
        (select mnist_train.y as actual, mnist_train_prediction.estimated_y as 
estimated 
         from mnist_train_prediction inner join mnist_train
         on mnist_train.id=mnist_train_prediction.id) q
    WHERE q.actual=q.estimated;
    
    2) Can you please add a comment at the top on the parameters you used to 
get a good accuracy + the platform you used?
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---