search

NeuroTechX / EEG-ExPy

EEG Experiments in Python
https://neurotechx.github.io/EEG-ExPy/
BSD 3-Clause "New" or "Revised" License
447 stars 125 forks source link

syntax error in muselsl #277

Open danielemarinazzo opened 1 month ago

danielemarinazzo commented 1 month ago

ℹ Computer information

📝 Provide detailed reproduction steps (if any)

We installed everything on a Mac M2 OS 14.3.1 (23D60)

✔️ Expected result

running eegnb -ip and go to the next step

❌ Actual result

when running eegnb runexp -ip (and when running the test cell as in the installation guide) we get this 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))

Duplicate from https://github.com/alexandrebarachant/muse-lsl/issues/216#issue-2578472803

JohnGriffiths commented 1 month ago

hi Daniele.

1 - can you share a screenshot of the exact terminal commands and output

2 - recommended device type to use is muse2_bfn, which uses brainflow not muselsl as backend.

danielemarinazzo commented 1 month ago
GeplakteAfbeelding-1

How to make muse2_bfn default? A previous error complained about brainflow not being installed

BoBo262 commented 1 month ago

Ran Into the same problem. Tried to run the installation test but got a SyntaxError related to muselsl Screenshot_3

ErikBjare commented 1 month ago

That syntax (the "walrus operator") is only supported in Python 3.8+

I'd suggest to drop support for Python 3.7, maybe even go all the way up to Python 3.10. (edit: nvm, I think we still have some dependencies through psychopy that makes this difficult)

BoBo262 commented 1 month ago

Thanks for the advice. I tried to create a new environment for the eeg notebook, this time in 3.8 but got a new error Screenshot_4

oreHGA commented 1 month ago

@BoBo262 , can you try the following and let us know what you get: