Skip to content

Conversation

ReneSkukies
Copy link
Member

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.

added a file that should (theoretically) help with easy creation of a noise artifact.
Delete testing artifact
@codecov-commenter
Copy link

Codecov Report

Merging #32 (51837e7) into main (4fef6bd) will not change coverage.
The diff coverage is n/a.

❗ 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           

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@behinger
Copy link
Member

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)

@behinger
Copy link
Member

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

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.

3 participants