Skip to content

Added BEP032 example dataset#429

Closed
TheChymera wants to merge 2 commits intobids-standard:masterfrom
TheChymera:bep032_data
Closed

Added BEP032 example dataset#429
TheChymera wants to merge 2 commits intobids-standard:masterfrom
TheChymera:bep032_data

Conversation

@TheChymera
Copy link
Copy Markdown
Contributor

@TheChymera TheChymera commented Mar 12, 2024

So as I submitted this pull request, I saw we already have a BEP032 branch. Is that the same thing but with a more up-to-date example? That one seems to be dealing with an eeg face perception dataset though 🤔

[deco]~/src/bids-examples ❱ git diff bep032 master --name-only | head -10
README.md
dataset_listing.tsv
eeg_ds003645s_hed/README
eeg_ds003645s_hed/dataset_description.json
eeg_ds003645s_hed/participants.json
eeg_ds003645s_hed/participants.tsv
eeg_ds003645s_hed/sub-002/eeg/sub-002_task-FacePerception_run-1_events.tsv
eeg_ds003645s_hed/sub-002/eeg/sub-002_task-FacePerception_run-2_events.tsv
eeg_ds003645s_hed/sub-002/eeg/sub-002_task-FacePerception_run-3_events.tsv
eeg_ds003645s_hed/sub-003/eeg/sub-003_task-FacePerception_run-1_events.tsv
[deco]~/src/bids-examples ❱ git diff bep032 master --name-only | rg ephys€
[deco]~/src/bids-examples ❱

@effigies

Co-authored-by: Julia Sprenger <julia.sprenger@rwth-aachen.de>
Co-authored-by: Yaroslav Halchenko <debian@onerussian.com>
@TheChymera
Copy link
Copy Markdown
Contributor Author

@yarikoptic couldn't just use your commits because:

decohost_2024-03-15T16:35:46

But there were other things to patch up as well.

@yarikoptic
Copy link
Copy Markdown
Contributor

never saw such a thing and never had problem committing from github! Were you able to provide/commit suggestions in the past? Please check that you have correct email set in your profile. See https://docs.github.com/en/account-and-profile/setting-up-and-managing-your-personal-account-on-github/managing-email-preferences/adding-an-email-address-to-your-github-account

@TheChymera
Copy link
Copy Markdown
Contributor Author

@effigies sorry for pinging you again, but could you clarify what — if anything — is going on on the other bep032 branch? Or do you know who might be able to tell me? It doesn't look like bep032 at all unless I'm missing something.

@effigies
Copy link
Copy Markdown
Contributor

The existing bep032 branch appears to have been created last month (Feb 14) by @robertoostenveld.

https://github.com/bids-standard/bids-examples/branches

It is behind master. I assume the goal was to start to push things to it, but I don't know who's involved in that effort.

@TheChymera
Copy link
Copy Markdown
Contributor Author

@robertoostenveld would you prefer to contribute here? this is a copy of the example data from here → https://gin.g-node.org/NeuralEnsemble/BEP032-examples

@robertoostenveld
Copy link
Copy Markdown
Collaborator

I made the bep032 branch here following discussion in the February BEP032 meeting; it is also mentioned in the "2024-02-14_INCF_WG_neuroscience_data_structure" minutes. The reason is that draft examples for the animal ephys cannot be merged yet in master as long as the BEP is not finalized, but we do want animal ephys examples from different contributors to appear side-by-side to allow checking for consistency etc. The shared bep032 branch allows for that.

I have just updated the bep032 branch to the latest master and have merged your bep032_data into it. I will create a draft pull request for bep032 into master to replace this one.

@TheChymera
Copy link
Copy Markdown
Contributor Author

@robertoostenveld but there are other diffs in that branch which to me at least seem BEP032 unrelated, as mentioned in the original post:

[deco]~/src/bids-examples ❱ git diff bep032 master --name-only | head -10
README.md
dataset_listing.tsv
eeg_ds003645s_hed/README
eeg_ds003645s_hed/dataset_description.json
eeg_ds003645s_hed/participants.json
eeg_ds003645s_hed/participants.tsv
eeg_ds003645s_hed/sub-002/eeg/sub-002_task-FacePerception_run-1_events.tsv
eeg_ds003645s_hed/sub-002/eeg/sub-002_task-FacePerception_run-2_events.tsv
eeg_ds003645s_hed/sub-002/eeg/sub-002_task-FacePerception_run-3_events.tsv
eeg_ds003645s_hed/sub-003/eeg/sub-003_task-FacePerception_run-1_events.tsv

Is something else going on in parallel on that branch?

@robertoostenveld
Copy link
Copy Markdown
Collaborator

Hi @TheChymera I am not sure what changes you are referring to, on https://github.com/bids-standard/bids-examples/pull/430/files I only see your additions in the ephys_BEP03 directory. I think that in the comparison above you are also showing unrelated changes that have happened on the master branch.

@TheChymera
Copy link
Copy Markdown
Contributor Author

TheChymera commented Apr 4, 2024

@robertoostenveld yes, you are right, I was misusing git diff. Let's move to the bep032 branch then.

@TheChymera TheChymera closed this Apr 4, 2024
@TheChymera TheChymera deleted the bep032_data branch April 4, 2024 21:13
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.

4 participants