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

build: moved to pyproject.toml/poetry #190

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

ErikBjare
Copy link
Collaborator

I have no idea how, but I managed to close the previous PR with a force push somehow (#22)... So here we go again.

ci: moved to using poetry for docs CI, placeholder test for pytest

build: updated mne dependency constraint to >=0.20.8
@ErikBjare ErikBjare force-pushed the dev/poetry branch 2 times, most recently from b22ee78 to b88c0c7 Compare June 22, 2022 21:58
@JohnGriffiths
Copy link
Collaborator

Note 1: New poetry release will have formal notion of dependency groups, consider using this in eegnb for separating a) streaming, b) stim pres, c) analysis ?

Note 2: Ore desperately wants to have this working for new Macs specifically. Use OS specific options here with that?

Parvfect and others added 3 commits September 30, 2022 10:09
…ific experiments to subclasses (NeuroTechX#184)

* First commit

* Second commit

* Modifications

* Lol

* Lol

* Incorporated N170 and p300, looking good for a PR

* ssvep update

* Implementing subclasses instead of loose functions

* fix: fixed import (brainflow updated API)

* Playing around still

* Fixing import errors

* Adding abstractmethod decorators

* Still working on the import error

* Guess what's finally working

* Comments and naming ticks

* More comments

* Live coding demonstration

* ssvep adapted

* Adapting Auditory Oddball

* changing save_fn to self.save_fun

* This maybe the last big change

* utils file changed, changes work through cli

Co-authored-by: Erik Bjäreholt <erik@bjareho.lt>
@ErikBjare
Copy link
Collaborator Author

I have a Macbook Air M2 to test on now too, so giving this another shot. Hoping it will also fix CI.

@ErikBjare
Copy link
Collaborator Author

Good news! Looks like wxPython finally pushed wheels to PyPI: https://pypi.org/project/wxPython/4.2.0/#files

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

Successfully merging this pull request may close these issues.

None yet

3 participants