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.

Sign up for GitHub

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

Jump to bottom

Merge in the preprocessing pipeline #88

Closed
wants to merge 76 commits into from
Closed

Merge in the preprocessing pipeline #88

wants to merge 76 commits into from

Conversation

caiw
Copy link
Member

@caiw caiw commented Dec 13, 2023 •
edited
Loading

Fixes #81.

neukym added 30 commits October 14, 2016 19:45
@neukym
Initial commit
910945e
@neukym
First commit of code from MRC-CBU
97316c2
@neukym
Create .gitignore
368e50b
@neukym
Updates pipeline to remove depricated versions
dcced0d
@neukym
Starts migration to Python
9abd089
@neukym
Updates readme and gitignore
4e6fdd3
@neukym
Update preprocessing.py
2e32d1a
@neukym
Adds Maxwell Filtering
5a4db37
@neukym
Minor code cleanup
dc14588
@neukym
Adds first draft of epoch cutting
a83cf7d
@neukym
Adds option for ICA eyeblink removal
76af5cd
@neukym
Update aa_improvements_to_add_in.docx
5fbeee2
@neukym
Corrects maxfilter bug
2d2e7c7
@neukym
Improves notch filtering
fa51964
@neukym
Adds looping and bad channels from YAMLs
db055ee
@neukym
Updates preprocessing
a90cd56
@neukym
Adds evoked averaging
3a9f8c2
@neukym
Minor update to config
144e9ae
@neukym
Sorts legacy files
c65ceb2
@neukym
Moves format to 'short' to save space
1a82467
@neukym
Adds graphic to readme.md
9f00e97 
Fixes #1
@neukym
Update README.md
173288b
@neukym
Update dataset4_config_file.yaml
0c597a8
@neukym
Update preprocessing.py
f754ef6
@neukym
mne update package version
e06a38a
@neukym
adds in new dataset config
ea217ac
@neukym
Puts equiptment latency checker in the right place
710ed48
@neukym
Corrects incorrect audio event finding
125eb01
@neukym
Adds automatic head position extraction
c4a10f2
@neukym
Adds audio_delivery_shift_correction
0e03dbd
@caiw caiw self-assigned this Dec 13, 2023
@caiw caiw linked an issue Dec 13, 2023 that may be closed by this pull request
Merge kymata-preprocessing-pipeline in #81
Closed
5 tasks
@caiw caiw added ❓ discussion needed Extra discussion is needed before work can commence 👷 design standards labels Dec 13, 2023
@caiw
Copy link
Member Author

caiw commented Dec 13, 2023

This is blocking #89 and #76, so I suggest we make this a bare-bones merge, deciding roughly how it should look, and then refine it with tests, demos etc, as separate issues/PRs

@caiw caiw mentioned this pull request Dec 13, 2023
Merged
@neukym
Copy link
Member

neukym commented Dec 13, 2023

Sounds good to me. Let's discuss on Friday 👍

@caiw caiw removed the ❓ discussion needed Extra discussion is needed before work can commence label Dec 15, 2023
@caiw caiw closed this Dec 15, 2023
@caiw caiw deleted the preprocessing-combine branch December 15, 2023 12:21
@caiw
Copy link
Member Author

caiw commented Dec 15, 2023 •
edited
Loading

Moving this to #91 and #92 for easier review, sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@caiw caiw

Labels
👷 design standards preprocessing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Merge kymata-preprocessing-pipeline in
3 participants
@caiw @neukym @young-x-skyee