Closed seventeen-iterator closed 5 years ago
Hello @seventeen-iterator good work so far. However, most of the code had been covered in the previous PR right, so we would want something more novel.
Here's what we can try. Write code for sample usage for your own neural network model. You can create a simple CNN for instance using instructions in the same blog post.
Do the above in a separate python file. Update TODO in your current python file saying "Once graph export it is fixed, usage functionality from the new file can be inserted here".
Hello, I added CNN that trained on the fashion MNIST dataset. I'm not sure how good is the model I created, but more than enough for a sample.
Hello @seventeen-iterator I have made some suggestions on your submission. Check it out.
Also added some progress bars to make it look nearly like keras
Is this easy enough to understand now?
Hello @seventeen-iterator, code looks good. CLI logger also looks good. I'm attaching a screenshot here for other mentors reference:
Two things:
from keras.datasets import fashion_mnist
for python2. Works for python3 though. Did you encounter such an issue?tensorflow_pbtext_usage.md
along with a description of what it does. Don't forget to mention users to pip install tqdm
@thatbrguy Hi! I wasn't launching it first, because I didn't have python2. Also tensorflow doesn't exist for Windows + Python2. But I managed to launch tensorflow_fashion_mnist.py
on the virtual machine, and no errors caught. (I installed anaconda, tensorflow and tqdm first)
The rest I'll satisfy later.
@seventeen-iterator Oh you're using windows. Alright then. Address the other changes before your time runs out! (I believe you have 5 hours or so left). You can request for additional time if needed.
Both lines executed without errors in python2.
Added another documentation about usage of exported models in tensorflow.