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 in muselsl #277

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

syntax error in muselsl #277

danielemarinazzo opened this issue Oct 10, 2024 · 6 comments
Labels
bug Something isn't working

Comments

@danielemarinazzo
Copy link
Contributor

ℹ Computer information

  • Platform OS (e.g Windows, Mac, Linux etc): MAC M2
  • Python Version: 3.7
  • Brain Interface Used (e.g Muse, OpenBCI, Notion etc): Muse

📝 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 alexandrebarachant/muse-lsl#216 (comment)

@danielemarinazzo danielemarinazzo added the bug Something isn't working label Oct 10, 2024
@JohnGriffiths
Copy link
Collaborator

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
Copy link
Contributor Author

GeplakteAfbeelding-1

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

@BoBo262
Copy link

BoBo262 commented Oct 18, 2024

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

@ErikBjare
Copy link
Collaborator

ErikBjare commented Oct 18, 2024

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
Copy link

BoBo262 commented Oct 18, 2024

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
Copy link
Collaborator

oreHGA commented Oct 21, 2024

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

  • creating the eeg-expy environment without wxpython
    conda create -n "eeg-expy" python=3.8 git pip

  • change directory to where eeg-expy is downloaded on your machine and then install
    cd <your_path_where_eegexpy_is_on_your_machine>
    pip install -e .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants