Shuffle sequence lengths #3926
Shuffle sequence lengths #3926
Conversation
shuffle with the rest of the data
use arma traits like the rest of the file
|
One thing I forgot to write, if you want to add a note to |
rcurtin
left a comment
rcurtin
left a comment
There was a problem hiding this comment.
Oh, sorry---forgot to kick off the build. It looks like one of the new tests is failing; can you look into that? Thanks!
|
not sure why that last test failed. my changes shouldn't affect it and all tests passed on my machine |
rcurtin
left a comment
rcurtin
left a comment
There was a problem hiding this comment.
Don't worry about the cross-compilation test; you're right that it has nothing to do with your code. Thanks again!
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
Thanks again @nzachary! |
3 participants
I noticed that rnns don't shuffle the sequence lengths with the data/labels
This pr should fix that