Ryan Curtin

Results 312 comments of Ryan Curtin

> For the ANN serialistion problem, there are a few approaches. First, I suggest making a simple define-based option to enable serialisation of ANNs, which is then detected by mlpack...

No worries, there are a few follow-up PRs in progress if you want to review those 😄 and, if you find any issues with this PR, I'll handle any comments...

This is worked around in #3200, but I agree, Go modules would probably be a better long-term fix. :+1:

Oops, mlpack-bot didn't listen :smile:

I like the idea, but maybe we should put this in a different repository? I don't think they fit in the base repository. There are some Dockerfiles in the `jenkins-conf`...

Fair enough---but I think it doesn't make sense to have CMake install them (so no need for any `install()` commands). How will we test these images / set up CI...

Thanks for explaining the general idea. It makes sense to me. Two additional questions though: - What should we do with the images in jenkins-conf? [here](https://github.com/mlpack/jenkins-conf/tree/master/Dockerfiles) Should we adapt our...

I think this one still needs to be handled, so I'll reopen it :+1:

Let's reopen this, I'll review it someday when I have a chance...

**Commented by rcurtin on 6 Apr 43643790 19:28 UTC** (In #300) You are right, if you use uint8_t, then it will take up about 6GB of RAM and then the...