index out of bounds -- in Data.Vector.Generic about grenade HOT 5 CLOSED

Wait on I'm a couple of commits behind. No I'm on the master head.

from grenade.

Can someone try and reproduce? ghc-8.2.2 / resolver: lts-10.2

from grenade.

G'day,

I bashed out the shakespeare example and included some not so safe code for decoding the results back into characters, which is probably what's biting you here.

Problems happens when there aren't the same number of unique characters as the length of the vector.

I trained it with
https://raw.githubusercontent.com/karpathy/char-rnn/master/data/tinyshakespeare/input.txt
so you might want to try that version of Shakespeare's texts.

The offending terrible code is in the https://github.com/HuwCampbell/grenade/blob/master/src/Grenade/Utils/OneHot.hs

If you're interested, it should be quite possible to abstract the network shape to a parameter i, then use a withSomeNat pattern to make this safe for any input.

from grenade.

Bit of a kludge here (a whole 30 seconds of work), but it should prevent the runtime error and allow people to adjust the size of the network to fit.

#49

from grenade.

Thanks Huw,
That gives me a leg up for generalising it for any input. I did get a problem in OneHot with some unrelated input so I'll give it a bash.

from grenade.