alexandrebarachant / muse-lsl

Python script to stream EEG data from the muse 2016 headset
BSD 3-Clause "New" or "Revised" License
629 stars 184 forks source link

[MRG] Changed matplotlib backend to PyQt5Agg #82

Closed Hororohoruru closed 5 years ago

Hororohoruru commented 5 years ago

Takes care of issue #81, but I'm not sure it is a common issue

alexandrebarachant commented 5 years ago

What OS are you using ? I'm not sure this is the ideal solution, and it might break others workflow.

Hororohoruru commented 5 years ago

I'm currently using Ubuntu 18.04. I agree with you, but since it is not an error thrown by the script, it might be difficult to handle correctly

alexandrebarachant commented 5 years ago

i'm rooting for an issues with dependencies. Last time i run the script on ubuntu I had no trouble with TkAgg. can you check online if other poeple ran into the same problem with matplotlib and see if this can be sove by updating a package ?

Hororohoruru commented 5 years ago

It doesn't seem to be any trend. I find ppl having the same issue with other backends, so usually they change backends to take care of it. Since it is a problem with matplotlib, this PR can be closed.

alexandrebarachant commented 5 years ago

One solution would be to add an extra argument so that you can switch backend. When you run the script.

On Fri, Mar 8, 2019, 9:59 AM Juan Jesús Torre Tresols < notifications@github.com> wrote:

It doesn't seem to be any trend. I find ppl having the same issue with other backends, so usually they change backends to take care of it. Since it is a problem with matplotlib, this PR can be closed.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/alexandrebarachant/muse-lsl/pull/82#issuecomment-470956987, or mute the thread https://github.com/notifications/unsubscribe-auth/ACC9tmNiSnaDOcfqnwGuIx8X4oz96KUYks5vUnrLgaJpZM4blcyX .

Hororohoruru commented 5 years ago

Good idea! on it

Hororohoruru commented 5 years ago

Anytime! Thanks for the feedback