Bandicoot library example to train an mnist example #219
Conversation
Signed-off-by: Omar Shrit <omar@avontech.fr>
Signed-off-by: Omar Shrit <omar@avontech.fr>
|
|
|
This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions! 👍 |
| * @author Omar Shrit | ||
| */ | ||
| #define MLPACK_ENABLE_ANN_SERIALIZATION | ||
| #define MLPACK_HAS_COOT |
There was a problem hiding this comment.
Just a TODO, we should integrate definition of MLPACK_HAS_COOT into the mlpack headers (maybe in prereqs.hpp or something, we try to detect Bandicoot), so that users don't have to write it by hand. You're probably already thinking that, I just wanted to write it down so it doesn't get forgotten.
| * @author Eugene Freyman | ||
| * @author Omar Shrit | ||
| */ | ||
| #define MLPACK_ENABLE_ANN_SERIALIZATION |
There was a problem hiding this comment.
Serialization will be a little tricky. When this macro is enabled, we compile all the serialize() functions for all the layer types---for the type arma::mat. Instead, we'll need to do CEREAL_REGISTER_MLPACK_LAYERS(coot::mat) or similar. Also, we'll need to actually implement a serialize() function for Bandicoot types, a bit like in src/mlpack/core/arma_extend/serialize_armadillo.hpp. I think the strategy should just be to convert it to an Armadillo matrix (i.e. pull it off the GPU into the CPU), and then serialize that, plus a little bit of code to put the Armadillo matrix back onto the GPU during loading. Anyway, I can help with that when we get there.
There was a problem hiding this comment.
Yes, for now, the easy solution for now would be to convert it back to Armadillo.
In all cases, we are loading an armadillo matrix, so we will have to do the conversion at least once.
Once we have the load functionality for Bandicoot, if it makes sense, then we can think of serialization.
|
This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions! 👍 |
|
needs to be reimplemented again, but I am re-opening it so I do not forget |
Hi,
This is a draft and still does not compile.