search

lina-usc / pylossless

🧠 EEG Processing pipeline that annotates continuous data
https://pylossless.readthedocs.io/en/latest/
MIT License
24 stars 8 forks source link

updated test file #44

Closed scott-huberty closed 1 year ago

scott-huberty commented 1 year ago

re-ran the pipeline on Face13/sub-s02 (the full recording) with our updated main branch. I tested loading the ouput to the dash qcr dashboard and it works.

These files are the saved output of pylossless

scott-huberty commented 1 year ago

the files changed tab is hard to read.

basically I replaced assets/test_data/sub-s02. I also just deleted assets/test_data/sub-s01 since we were never using it.

christian-oreilly commented 1 year ago

It could be good, tough, to have a second test file... even if it is just so that we can toggle from one to the next when testing stuff. But right now, s01 was broken anyway.

scott-huberty commented 1 year ago

It could be good, tough, to have a second test file... even if it is just so that we can toggle from one to the next when testing stuff. But right now, s01 was broken anyway.

I'll add another tomorrow, np.

scott-huberty commented 1 year ago

I added a second test file.

Note that these files are too large for us to permanently keep them in the repo. Before we deploy, we should address managing them using git-large-files or some other method.

Merging for now.