Open ReneSkukies opened 11 months ago
Merging #32 (51837e7) into main (4fef6bd) will not change coverage. The diff coverage is
n/a
.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@ Coverage Diff @@
## main #32 +/- ##
=======================================
Coverage 32.71% 32.71%
=======================================
Files 9 9
Lines 217 217
=======================================
Hits 71 71
Misses 146 146
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!
hey great! Have a look at https://github.com/unfoldtoolbox/UnfoldSim.jl/blob/multichannel/test/artifactheadmodel.jl to see how I uploaded the headmodel artifact for UnfoldSim (not yet merged)
regarding which EEG dataset to use. Something around 1h per subject, at least 40 subjects,
one thing to keep in mind is, that we basically want "clean" EEG datasets, that is without breaks. not sure how to achieve. any ideas?
We could also directly summon richard gao + his PhD to just generate us with plenty of (single electrode) EEG
I wrote a short script in a dev subfolder that should be able to easily add an artifact to the package. For now the artifact is lazily loaded, i.e. the artifact is not installed/ downloaded unless it is called upon. We would only need to change the url in the script, once we have uploaded a noise dataset as .tar file.
Real noise can then be used by calling on the artifact by using
artifact"RealNoise"
to get the artifact path and then loading it.