search

deepgram / kur

Descriptive Deep Learning
Apache License 2.0
814 stars 107 forks source link

Add CSV output via hooks #36

Closed ipeevski closed 7 years ago

ipeevski commented 7 years ago

Extend and enable CSV output (along with the previously existing pickle output). This makes it easier to look at raw data and analyze details further.

Based on discussion on gitter from Jan 27

ipeevski commented 7 years ago

Following up on the failed Travis checks ... it looks like it is unrelated to the changes I've made ... can you confirm or let me know what I can do to fix?!

ajsyp commented 7 years ago

Thanks for following up with the PR! Don't worry about the unit tests. Everything is a mess: Keras 2.0 got released today, but it isn't backwards compatible with Keras 1.x. As a result, everything is breaking left and right :) I'll be pushing out a compatibility update shortly that should fix it.

ipeevski commented 7 years ago

Oh, good. I was wondering what i managed to break in such a small patch :) And sorry for the long delay ... I need to get it together ;)

On Wed, 15 Mar 2017, 13:19 Adam Sypniewski, notifications@github.com wrote:

Thanks for following up with the PR! Don't worry about the unit tests. Everything is a mess: Keras 2.0 got released today, but it isn't backwards compatible with Keras 1.x. As a result, everything is breaking left and right :) I'll be pushing out a compatibility update shortly that should fix it.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/deepgram/kur/pull/36#issuecomment-286625550, or mute the thread https://github.com/notifications/unsubscribe-auth/AAIHrLAItbenyXKQNWVAzAxYTf0F4qJaks5rl1HIgaJpZM4MdZUG .