Skip to content

this missing import was preventing tests from running on my machine#457

Closed
jonathanstrong wants to merge 1 commit intoTheano:masterfrom
jonathanstrong:master
Closed

this missing import was preventing tests from running on my machine#457
jonathanstrong wants to merge 1 commit intoTheano:masterfrom
jonathanstrong:master

Conversation

@jonathanstrong
Copy link
Copy Markdown

No description provided.

@nouiz
Copy link
Copy Markdown
Member

nouiz commented Jun 20, 2017

What error this is fixing? Which version of nose do you use?

@jonathanstrong
Copy link
Copy Markdown
Author

jonathanstrong commented Jun 20, 2017

$ pip freeze | rg nose
nose==1.3.7
nose-parameterized==0.5.0

don't have the error handy but without this import I could not run the command DEVICE=cuda0 python -c "import pygpu; pygpu.test()"

@abergeron
Copy link
Copy Markdown
Member

abergeron commented Jun 21, 2017

This will fix the immediate problem, but there is the alternative problem that it triggers a flake8 warning.

#459 fixes the problem and avoids a flake8 warning.

@abergeron
Copy link
Copy Markdown
Member

This was addressed in #459 which was merged, but thanks for the fix.

@abergeron abergeron closed this Jun 21, 2017
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.

3 participants