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

Stop using the bundled six that comes with Theano. #1539

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abergeron
Copy link
Contributor

We want to remove the bundled copy in Theano so this makes pylearn2
compatible with that future change.

We want to remove the bundled copy in Theano so this makes pylearn2
compatible with that future change.
@lamblin
Copy link
Member

lamblin commented Jun 16, 2015

.requirements.txt should also be updated, I think. No need to update the "requirements" part of the doc since "dependencies of Theano" are already mentioned.

@nouiz
Copy link
Member

nouiz commented Jun 17, 2015

I'm not sure this is good as told in the python 2 and 3 PR in Theano.

On Tue, Jun 16, 2015 at 4:56 PM, Pascal Lamblin [email protected]
wrote:

.requirements.txt should also be updated, I think. No need to update the
"requirements" part of the doc since "dependencies of Theano" are already
mentioned.


Reply to this email directly or view it on GitHub
#1539 (comment).

Copy link

@BluePAZ BluePAZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is needed, and I had to change these files to get it to work with the latest Theano as well

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

Successfully merging this pull request may close these issues.

4 participants