Keras support about fabrik HOT 8 CLOSED

@virajprabhu I am interested in working on the issue. How do I start?

from fabrik.

Great! I'd recommend you set up the code locally if you haven't already (follow instructions in the README) and try out a few import/export workflows. The caffe_app/views directory contains the relevant code for caffe support. We will need a similar implementation for Keras (refer to my comment above). Once you've done that, let's have a discussion about the details of the implementation.

from fabrik.

@virajprabhu I have finished setting up the code for the project. For adding support for Keras, it would be similar to how it is done for tensorflow, correct?

from fabrik.

@souravsingh Great! Yes, the approach will be similar. Do you have a fair understanding of the TensorFlow logic?

from fabrik.

@virajprabhu I have a fair understanding of Tensorflow logic. Should I go ahead and make a WIP Pull Request to incorporate Keras Support?

from fabrik.

Great @souravsingh! Before you do that, we should iterate a bit over the implementation. Since this is a major feature, I'd like to make sure we are on the same page so you don't end up having to reimplement anything. Could you please summarize your proposed implementation here?

from fabrik.

@souravsingh PR #22 (under review) was just submitted for this. Is there any other issue you'd like to work on? #17 is another major feature that we're very interested in. Let me know if there is, and I could assign it to you.

from fabrik.

Closing this. #22 was merged to keras-support (Still in progress).

from fabrik.