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.

Sign up for GitHub

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

Download correct ddl for 64 system on windows with Plexon #1350

Merged
merged 9 commits into from
Dec 6, 2023
Merged

Download correct ddl for 64 system on windows with Plexon #1350

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
498a410
add architecture dependency in plexon
h-mayorquin Nov 17, 2023
0754b7b
Update neo/rawio/plexon2rawio/plexon2rawio.py
h-mayorquin Nov 17, 2023
e5ddfa3
add windows condition
h-mayorquin Nov 18, 2023
2e1b35e
scalar
h-mayorquin Nov 18, 2023
6737ecf
Update neo/rawio/plexon2rawio/plexon2rawio.py
h-mayorquin Nov 21, 2023
1b31b1a
Update neo/rawio/plexon2rawio/plexon2rawio.py
h-mayorquin Nov 21, 2023
bb29a5b
Update neo/rawio/plexon2rawio/plexon2rawio.py
h-mayorquin Nov 21, 2023
58266c7
remove warning
h-mayorquin Dec 6, 2023
e869c3b
Merge remote-tracking branch 'refs/remotes/origin/platform_neo' into …
h-mayorquin Dec 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove warning
  • Loading branch information
@h-mayorquin
h-mayorquin committed Dec 6, 2023
commit 58266c7d6cdc8b1b43231833a9c82b17644e553c
7 changes: 1 addition & 6 deletions neo/rawio/plexon2rawio/plexon2rawio.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,12 +86,7 @@ def __init__(self, filename, pl2_dll_file_path=None):
pl2_dll_folder.mkdir(exist_ok=True)
pl2_dll_file_path = pl2_dll_folder / file_name

if pl2_dll_file_path.exists():
# I think this warning should be removed
# Warnings should provide a solution but this is
# just a reminder to the user of normal behavior
warnings.warn(f'Using cached plexon dll at {pl2_dll_file_path}')
else:
if not pl2_dll_file_path.exists():
h-mayorquin marked this conversation as resolved.
Show resolved Hide resolved
h-mayorquin marked this conversation as resolved.
Show resolved Hide resolved
url = f'https://raw.githubusercontent.com/Neuralensemble/pypl2/master/bin/{file_name}'
dist = urlopen(url=url)

Expand Down