search

amplab / keystone

Simplifying robust end-to-end machine learning on Apache Spark.
http://keystone-ml.org/
Apache License 2.0
468 stars 116 forks source link

Samples GMM ref wasn't released in EncEval code #278

Closed Vaishaal closed 8 years ago

Vaishaal commented 8 years ago

I'm pretty sure this is what screwed me up last semester. Sigh.

shivaram commented 8 years ago

LGTM. The change looks good, but I'm not sure this can affect things much. The GMM is only run on the driver and only run once ?

Vaishaal commented 8 years ago

Oh well. Still worth fixing.

On May 29, 2016 12:59 PM, "Shivaram Venkataraman" notifications@github.com wrote:

LGTM. The change looks good, but I'm not sure this can affect things much. The GMM is only run on the driver and only run once ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/amplab/keystone/pull/278#issuecomment-222379597, or mute the thread https://github.com/notifications/unsubscribe/ABTEkAd9ILSGfp3Uk0vRfiAbs3EJjLWRks5qGfAfgaJpZM4IpW64 .