Alternatives to always-up-to-date submodules about distro HOT 8 CLOSED

@alexbw as I said, I am okay with either. I think we should give the one a try, i.e. keeping submodules at fixed commits. If we have any issues, we would uncover them as we go.
Historically, I built torch-distro for @jonathantompson, as he was complaining that he missed the old torch7-distro, and with torch7-distro, we just had one giant repo of everything (which was automatically on master by definition). But torch-distro has gotten bigger than that one single use-case.

Also, it's great that you are taking the lead on this, dont tone it down at all :)

from distro.

This is a great initiative. I'd vote for versioning commits, and upgrade them only when certain strict conditions are met (such as travis giving you the green light). It's especially important for /torch7, /nn, /cutorch and /cunn, where any subtle regression becomes a huge pain to trace.

from distro.

on board with that idea as well.

on the issue of travis, we have travis for torch7 and nn, but it almost always fails when running the unit tests, by just not responding for a long time. It seems to be a bug of VM + OpenBlas.

We also do not have travis testing for cutorch/cunn

If you guys have any ideas, let me know

from distro.

So about openblas, I think we should kill it for travis (if not everywhere...). This would speedup the build tremendously. What do you think?

For CUDA stuff, it's tricky to test for sure (no gpus on travis right?). Maybe we could host some of this testing somewhere else.

from distro.

Ok. I will remove the line in install.sh that automatically pulls all new repos and update all submodules to their current master branches.

After that, all new cool updates to included submodules have to be rolled in via a git submodule commit update. I think this is feasible, because if this distribution becomes a community standard, folks will pester us with github issues when it's time to update. I like the push better than pull model here :)

from distro.

Totally agreed!

from distro.

👍

from distro.

Done in dbf367a. Tested on laptop.

from distro.