Skip to content
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

Auto-update function? #71

Closed
hughperkins opened this issue Feb 18, 2016 · 4 comments
Closed

Auto-update function? #71

hughperkins opened this issue Feb 18, 2016 · 4 comments

Comments

@hughperkins
Copy link
Contributor

Two related observations:

  1. would be nice to be able to do torch-update, and it will update at least torch7, nn, cutorch, cunn, to latest version
  2. would be nice if installing distro automatically installed the latest versions :-) Currently I think that the version are set in the git submodules? And so we dont always get the latest versions, and have to build twice?
@soumith
Copy link
Member

soumith commented Feb 18, 2016

installing distro installs the most recent stable versions. everytime I update distro, I run the full suite of tests to make sure everything's okay. distro is for user-deployment, and hence this is suited.
i'm introducing named-versions soon.

@hughperkins
Copy link
Contributor Author

Hmmm... well.... so....a problem I face is, eg with the THNN changes, distro hasnt changed yet, but clnn will have changed, following a merge, so we get the scenario:

  • someone installs with distro (gets the old version of nn, pre-merge)
  • then they install cltorch, clnn, which isnt currently part of distro, and get the new, post-merge version

Of course, in this scenario, clnn doesnt work.

@hughperkins
Copy link
Contributor Author

So, there seem to be a few options for this:

  1. leave it as it is. I deal with the upgrade bugs as and when they occur
  2. make it easy for people to update to latest cutting-edge version of at least torch, nn, eg by providing a torch-upgrade script in distro.
  3. add cltorch and clnn to distro :-)

I think option 1 is not ideal. Option 3 is borderline perhaps. Option 2 seems relatively easy :-)

@hughperkins
Copy link
Contributor Author

Closing this, and reopening in the opposite direction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants