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

Add noise artifact #32

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add noise artifact #32

wants to merge 2 commits into from

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