rdadolf / fathom

Reference workloads for modern deep learning methods.
Apache License 2.0
73 stars 21 forks source link

Running with TF 1.0 #26

Closed rajivkapoor closed 7 years ago

rajivkapoor commented 7 years ago

I read the paper (great read BTW!) and wanted to try the workloads out but ran into some basic issues. I am a newbie to TF/ML but it looks like the TF scripts are not compatible with TF 1.0? If yes then is there any plan to make them work on TF 1.0?

rdadolf commented 7 years ago

Hi there. Glad you enjoyed the read.

Yes, it's something I'd like to have updated. Currently, there is a team from IBM and a friend here at Harvard who I believe both have versions working. I believe I should be getting a pull request for this (and a bunch of other refactoring) in the upcoming week. No promises on an exact date, but it's in the works.

Thanks for the interest, and I'll make sure to update and close this issue when the changes are incorporated.

rajivkapoor commented 7 years ago

Good to hear that. Looking fwd to it. Thanks!

rajivkapoor commented 7 years ago

Just a gentle ping .... any update? :)

svrama commented 7 years ago

Merged the upgrades from @Zzzoom in #27.

A few of the workloads still need some cleanup in TF 1.0.x, so let us know if you run into any issues.

rajivkapoor commented 7 years ago

Thanks! Will give it a shot here soon.