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

Rename aux.py to auxiliary.py to make pynn works on Windows machines. #678

Merged
merged 1 commit into from
Aug 18, 2020

Conversation

ChihweiLHBird
Copy link
Contributor

@ChihweiLHBird ChihweiLHBird commented Aug 17, 2020

Since aux is a reserved file name on Windows, aux.py is not allowed on Windows machine and can lead to an error. Simply change the name of the file from aux.py to auxiliary.py can fix it.

Microsoft Documents: https://docs.microsoft.com/de-de/windows/win32/fileio/naming-a-file
Similar change done for another python package: TUD-RST/symbtools#5
Related issues: #666, #446

@coveralls
Copy link

Coverage Status

Coverage remained the same at 43.644% when pulling d190ce4 on ChihweiLHBird:master into d8056fa on NeuralEnsemble:master.

@ChihweiLHBird
Copy link
Contributor Author

Python 2.7 and 3.3 tests failed but Python 3.6 test works fine. Since 2.7 and 3.3 are out of official support period, I would suggest removing Python 2.7 and 3.3 and add 3.5, 3.7, 3.8 for testing.

@apdavison
Copy link
Member

The test failure for 2.7 is one which is known to fail occasionally as it involves a statistical test; it has nothing to do with the PR. The failure for 3.3 is in uploading coverage stats, the tests actually all pass.

(Agreed on updating the test matrix to more recent versions of Python; we plan to do this soon).

@apdavison apdavison merged commit 1095983 into NeuralEnsemble:master Aug 18, 2020
casperdcl added a commit to AMYPAD/AmyPET that referenced this pull request Nov 25, 2022
@macknight
Copy link

install tf-encrypted failed. how to rename? I can't even find the fold.

ERROR: The tar file (C:\Users\xx\AppData\Local\Temp\pip-unpack-plha6gd0\tf-encrypted-0.9.1.tar.gz) has a file (C:\Users\xx\AppData\Local\Temp\pip-install-bj8xrdtv\tf-encrypted_08c1f7c5ab8c46a19975b7aaec907caf\tf_encrypted/operations/aux) trying to install outside target directory (C:\Users\xx\AppData\Local\Temp\pip-install-bj8xrdtv\tf-encrypted_08c1f7c5ab8c46a19975b7aaec907caf)

@codenum137
Copy link

@macknight
hi, I get into the same problem when installing tf-encrypted, have you fixed it? how? thx

@apdavison
Copy link
Member

apdavison commented Apr 29, 2024

@macknight @codenum137 I don't think your issue is relevant to PyNN

@NeuralEnsemble NeuralEnsemble locked as off-topic and limited conversation to collaborators Apr 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants