You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2018/09/20 16:22:59 UTC

[GitHub] kjchalup commented on issue #11884: im2rec documentation is lacking / buggy

kjchalup commented on issue #11884: im2rec documentation is lacking / buggy
URL: https://github.com/apache/incubator-mxnet/issues/11884#issuecomment-423246008
 
 
   Thank you for the response!
   
   1) The page you linked has a nice example thanks! It'd be great if it was discoverable from [this page](https://mxnet.incubator.apache.org/architecture/note_data_loading.html).
   
   2) im2rec.py still has no docstrings, nor even a line of comments besides the license boilerplate :( It's really not trivial for someone who has no idea how im2rec works to figure out what is going on in there quickly.
   
   3) There's a slight misunderstanding. As I said in the original post, I tried to make my own .lst file *by hand*, following the example on the [doc page](https://mxnet.incubator.apache.org/faq/recordio.html) which at the time had integer labels. I wasn't using im2rec.py, as I gave up on understanding it, lacking an example and any docstrings/comments. I was just using Vim to write stuff :) This didn't work with mx.io.ImageRecordIter, because the labels must in fact be floats, not ints. However, the docs are now updated -- the labels are floats. So the problem is fixed :) Thanks!

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services