search

bstriner / keras-tqdm

Keras integration with TQDM progress bars
MIT License
348 stars 41 forks source link

order, minimal use case, no tqdm intro #4

Closed stared closed 7 years ago

stared commented 7 years ago

I made some changes, see if you like them.

First, I wanted to give minimal example (it was not clear if I need to import tqdm separately).

Second, I wanted to start from context (Keras, tqdm). Also, instructions on pure tqdm should be (IMHO) left to tqdm, otherwise they are a distraction.

Third, I changed order of a few things.

I didn't change (though, I had a temptation to) TQDM to tqdm (official name, consistently written in lower case). I suggest to use it lower cased and consistently refer to this package as keras-tqdm, to minimize confusion. (But it's my aesthetics, I know you may disagree.)

(I changed in a way I would have benefited from it the most.)

In any case - today I am giving a talk about Keras at a Meetup (>200 participants) and will use your library, because it's cool! Thanks again! :)

bstriner commented 7 years ago

@stared Sorry I didn't get a chance to get to this earlier. Looks good and well written. Thanks!

Pushed out version 1.0.7 to PyPI with the new docs and new defaults.

Had to do a little editing because PyPI RST is apparently different than Github RST. Now, it is displaying nicely on both github and pypi. Take a look:

https://pypi.python.org/pypi/keras-tqdm

Cheers, Ben