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

WIP: Outline of model integration + preprocessing #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

minneker
Copy link
Collaborator

Work in progress. Preprocessing includes low-pass filtering, notch filtering and decimation. Model needs to dynamically change input size based on user inputs. I suggest we restrict user input for bin size so we can anticipate a couple of fixed input sizes. The reason this is tricky is because the architecture of EEGNet depends on the number of samples.

The main item necessary for integration is loading .edf file in the backend. It is unclear to me how to read a binary string as an .edf file (haven't found package functionality for that), and I'm thinking we may have to pass it to the backend another way since we rely heavily on mne in the backend.

@saketh-n
Copy link
Owner

saketh-n commented May 20, 2021

Makes sense Rob. edf issue is resolved, as for samples and model architecture we could change the architecture to only take one to n samples and run many models in parallel

EEG_NET seems to take variable # of samples, but I'll look into it further

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.

2 participants