-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Float -- episode 1 #3534
Float -- episode 1 #3534
Conversation
Signed-off-by: Omar Shrit <omar@avontech.fr>
Signed-off-by: Omar Shrit <omar@avontech.fr>
Signed-off-by: Omar Shrit <omar@avontech.fr>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, looks good to me. 👍 I'm sure there will be more work we have to do for the bandicoot adaptation, but getting arma::fmat
to work throughout is a great first step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Second approval provided automatically after 24 hours. 👍
For posterity, this is related to #290 |
This PR allows to compile mnist_simple with
arma::fmat
instead of the standardarma::mat
@rcurtin feel free to review, I only tested mnist_simple. If necessary for the other will be a new episode of Float PRs
The duck is helpful with these complex errors, it took much less time than expected this time.