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

syntax error #216

Open
danielemarinazzo opened this issue Oct 10, 2024 · 4 comments
Open

syntax error #216

danielemarinazzo opened this issue Oct 10, 2024 · 4 comments

Comments

@danielemarinazzo
Copy link

Mac M2 OS 14.3.1 (23D60)

when running "muselsl" command we get a syntax error message (corresponding to the colon after "device")

File "/Users/USERNAME/miniconda3/envs/eeg-notebooks/lib/python3.7/site-packages/muselsl/muse.py", line 90
if ((device := self.adapter.connect(self.address, retries))

@ErikBjare
Copy link
Contributor

ErikBjare commented Oct 18, 2024

That syntax requires Python 3.8+: NeuroTechX/EEG-ExPy#277 (comment)

The incompatibility with Python 3.7 was introduced in this recent commit: eb92e1a

@danielemarinazzo
Copy link
Author

Thanks! So the official installation instructions should mention 3.8? Or will something else break if we use that version?

@ErikBjare
Copy link
Contributor

ErikBjare commented Oct 18, 2024

I think the syntax should be changed to support Python 3.7 again since that's trivial.

Not up-to-date with things in EEG-ExPy, but I think it might be pretty tied to Python 3.7 still due all the shenanigans in PsychoPy: https://www.psychopy.org/download.html#linux

@danielemarinazzo
Copy link
Author

Ok, thanks. So there's a solution to use the package as it is? If necessary bypassing muselsl and using brainflow only (even iv on some laptops only the former works for some reason)?

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

No branches or pull requests

2 participants