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

Ensure 3.5 compatibility #9

Merged
merged 2 commits into from Mar 27, 2017
Merged

Ensure 3.5 compatibility #9

merged 2 commits into from Mar 27, 2017

Conversation

JasonKessler
Copy link
Contributor

It looks like Python 3.5 requires the module names to be dotted. Otherwise, it throws an error

...
from Porter import Porter
    ImportError: No module named 'Porter'

It looks like Python 3.5 requires the module names to be dotted.  Otherwise, it throws an error

```
...
from Porter import Porter
    ImportError: No module named 'Porter'
```
@nok
Copy link
Owner

nok commented Mar 13, 2017

Hello @JasonKessler,

thanks for the hint. Seriously I'm surprised about this error, because all tests passed successfully:
https://travis-ci.org/nok/sklearn-porter/jobs/209585865#L121

Before the next release/update I will test this issue in a local test environment. Because of that I keep this issue and pull request open.

Happy coding,
Darius 🌵

@JasonKessler
Copy link
Contributor Author

Thanks, Darius. It surprised me as well. It's possible it's just a quirk of my Anaconda 3.5 environment.

@nok nok added the bug label Mar 15, 2017
@nok nok merged commit b5ca039 into nok:master Mar 27, 2017
@nok
Copy link
Owner

nok commented Mar 27, 2017

Thanks @JasonKessler for your support!

Best,
Darius

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

Successfully merging this pull request may close these issues.

None yet

2 participants