EEG-ExPy
EEG-ExPy copied to clipboard
NeuroTechX
Experiment class refactor
Defined General Experiment Class type that does all of the operations besides loading and presenting the stimulus which is overriden by the derived classes. As of now it incorporates the Visual N170 and Visual P300 experiments - and working towards incorporating Visual SSVEP.
Some things worth thinking about, for the present_stimulus function, a lot of parameters are being passed which is annoying me, so I want to know if it would be smarter to pass the Experiment as an object instead (although I know in general that's bad practice). It also gives us a lot of flexibility for the future complex experiements.
Additionally, since all the main functions are delegated to lower classes, this mechanism should allow a lot of complexity and specificity and generally think this should work (and it looks cool too!).
Has not been tested besides compiling, so that needs to be done once whilst I work towards incorporating the rest of the experiments.
Has not been tested besides compiling, so that needs to be done once whilst I work towards incorporating the rest of the experiments.
I think you should definitely focus on one or two experiments and get them working before you start modifying the rest.
I also just fixed so that CI runs on PRs targeting develop, so if you rebase/merge in those changes you should get CI to run.