Updating the bep009 with the latest master branch updates #576
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
=== Do not change lines below ===
{
"chain": [],
"cmd": "git-sedi suffices suffixes",
"exit": 0,
"extra_inputs": [],
"inputs": [],
"outputs": [],
"pwd": "src/schema"
}
^^^ Do not change lines above ^^^
…terials using schema
also added a stab for entity_table beginning -- just had to RF loading first
(git)lena:~/proj/bids/bids-specification[ref/json-entity]git $> tools/bids-schema entity-table | Entity | Subject | Session | Task | Acquisition | Contrast Enhancing Agent | Reconstruction | Phase-Encoding Direction | Run | Corresponding Modality | Echo | Recording | Processed (on device) | Space | |------------------------------------------------------------------------------------------------|---------------|---------------|----------------|---------------|----------------------------|------------------|----------------------------|---------------|--------------------------|----------------|---------------------|-------------------------|-----------------| | Format | `sub-<label>` | `ses-<label>` | `task-<label>` | `acq-<label>` | `ce-<label>` | `rec-<label>` | `dir-<label>` | `run-<index>` | `mod-<label>` | `echo-<index>` | `recording-<label>` | `proc-<label>` | `space-<label>` | | anat<br>(T1w T2w T1rho T1map T2map T2star FLAIR FLASH PD PDmap PDT2 inplaneT1 inplaneT2 angio) | REQUIRED | OPTIONAL | | OPTIONAL | OPTIONAL | OPTIONAL | | | | | | | | | anat<br>(defacemask) | REQUIRED | OPTIONAL | | OPTIONAL | OPTIONAL | OPTIONAL | | | OPTIONAL | | | | | | beh<br>(stim physio) | REQUIRED | OPTIONAL | REQUIRED | | | | | | | | | | | | beh<br>(events beh) | REQUIRED | OPTIONAL | REQUIRED | | | | | | | | | | | | dwi | REQUIRED | OPTIONAL | | OPTIONAL | | | OPTIONAL | OPTIONAL | | | | | | | dwi<br>(sbref) | REQUIRED | OPTIONAL | | OPTIONAL | | | OPTIONAL | OPTIONAL | | | | | | | eeg | REQUIRED | OPTIONAL | REQUIRED | OPTIONAL | | | | OPTIONAL | | | | | | | fmap<br>(phasediff phase1 phase2 magnitude1 magnitude2 magnitude fieldmap) | REQUIRED | OPTIONAL | | OPTIONAL | | | | OPTIONAL | | | | | | | fmap<br>(epi) | REQUIRED | OPTIONAL | | OPTIONAL | OPTIONAL | | REQUIRED | OPTIONAL | | | | | | | func<br>(bold cbv phase sbref) | REQUIRED | OPTIONAL | REQUIRED | OPTIONAL | OPTIONAL | OPTIONAL | OPTIONAL | OPTIONAL | | OPTIONAL | | | | | func<br>(events) | REQUIRED | OPTIONAL | REQUIRED | OPTIONAL | OPTIONAL | OPTIONAL | OPTIONAL | OPTIONAL | | OPTIONAL | | | | | func<br>(physio stim) | REQUIRED | OPTIONAL | REQUIRED | OPTIONAL | | OPTIONAL | | OPTIONAL | | | OPTIONAL | OPTIONAL | | | ieeg | REQUIRED | OPTIONAL | REQUIRED | OPTIONAL | | | | OPTIONAL | | | | | | | meg | REQUIRED | OPTIONAL | REQUIRED | OPTIONAL | | | | OPTIONAL | | | | OPTIONAL | | | meg<br>(headshape) | REQUIRED | OPTIONAL | | OPTIONAL | | | | | | | | | OPTIONAL | | meg<br>(markers) | REQUIRED | OPTIONAL | OPTIONAL | OPTIONAL | | | | | | | | | OPTIONAL | | channels<br>(meg eeg ieeg) | REQUIRED | OPTIONAL | REQUIRED | | | | | OPTIONAL | | | | | | | electrodes<br>(eeg ieeg) | REQUIRED | OPTIONAL | | OPTIONAL | | | | | | | | | OPTIONAL | | events<br>(meg eeg ieeg) | REQUIRED | OPTIONAL | REQUIRED | | | | | OPTIONAL | | | | | | | photo<br>(meg eeg ieeg) | REQUIRED | OPTIONAL | | OPTIONAL | | | | | | | | | |
venvs -- virtualenvs I typically have around .idea -- pycharm configuration
=== Do not change lines below ===
{
"chain": [],
"cmd": "tools/generate_entry_table",
"exit": 0,
"extra_inputs": [],
"inputs": [],
"outputs": [],
"pwd": "."
}
^^^ Do not change lines above ^^^
One big change is that it merges rows within datatypes that have the same entities.
This is primarily to make it easier to run things within Python, since click-decorated functions can’t be called like Python functions.
* Fix typos in DECISION-MAKING file. * Add self to contributors. * Add utility contributions too. * Revert contributors table spacing changes.
Co-authored-by: Satrajit Ghosh <[email protected]>
Co-authored-by: Chris Markiewicz <[email protected]>
[INFRA] enable pandoc emojis for the pdf build
…r EEG, whcih also makes it acceptable for MEG and iEEG. See #561
[ENH] added PPG as an accepted channel type for EEG, MEG and iEEG
melanieganz
requested review from
chrisgorgo,
CPernet,
effigies,
monkeyman192 and
sappelhoff
as code owners
|
I pushed master to bep-009. There were no changes, so it went in clean. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[MISC] I would just like to update the bep-009 branch that was made while ago with the latest updates from master before we add our actual changes and make a pull request.