bids-specification
bids-specification copied to clipboard
[ENH] microelectrode electrophysiology specification (BEP032)
Replaces https://github.com/bids-standard/bids-specification/pull/1352 submitted from a fork outside of bids-specification.
Add specification for microelectrode electrohpysiology datasets based on the BEP032 proposal
[!Note]
We meet regularly and everyone is welcome
Next meeting: insert date on URL to join
Communication channel: https://framalistes.org/sympa/info/neuroscience-data-structure
[!Tip]
- To use this WiP schema on sample datasets, see https://deno.land/x/[email protected]#modifying-and-building-a-new-schema on how to use "stock"
bids-validator
with a custom schema. (attn @TheChymera)
TODOs
- [x] Add (at least one) .md with the section for the added new modality/datatype
- [x] #1800
- [ ] Please ensure your name is credited on our Contributors appendix. To add your name, please edit our Contributors wiki and add your name with the type of contribution. For assistance, please tag @bids-standard/maintainers.
- [ ] After opening the PR, our continuous integration services will automatically check your contribution for formatting errors and render a preview of the BIDS specification with your changes.
To see the checks and preview, scroll down and click on the
show all checks
link. From the list, select theDetails
link of theci/circleci: build_docs artifact
check to see the preview of the BIDS specification. - [ ] Add instructions here on how to run new
bids-validator
using schema in this PR- Currently there is ongoing effort in https://github.com/bids-standard/bids-examples/pull/435 to assess the state of deno based validator against examples, then #1798 is the first one trying it on whitelisted set of packages, and I think we should create a helper action for that : https://github.com/bids-standard/bids-validator/issues/1931
- [ ] Populate schema with specifications from the google doc ...
- [ ] The majority of changes: https://github.com/bids-standard/bids-specification/pull/1826
- [ ] Further markdown description: @Peyman-N is working on a PR
- [ ] Define enums of coord spaces to be added -- some image based, some ad-hoc
- [ ] Add CI action (likely github) to run
bids-validator
on sample datasets and this modified schema (@yarikoptic)
Issues this PR would likely to address
- Fixes #1375
Issues to see being addressed while working on this BEP (likely to move above) or not (moved below):
- #1634
- #1481
Other issues which relate but not in scope here and provided for reference/backreference
- #1669
- #1133
- #1165
- #197
- https://github.com/bids-standard/bids-2-devel/issues/54
Also relevant if you'd like to comment. → https://docs.google.com/document/d/1oG-C8T-dWPqfVzL2W8HO3elWK8NIh2cOCPssRGv23n0/edit?disco=AAABIzHGpUU
Also I forgot to link to this when I wrote it → https://docs.google.com/document/d/1oG-C8T-dWPqfVzL2W8HO3elWK8NIh2cOCPssRGv23n0/edit?disco=AAABIGPAMOw
Random suggestion: maybe having a markdown file in the source tree and the mkdoc config so you can at least use a macro to render the filename template.
The HTML version of this could then be reviewed by people still working on the google doc: may help if you don't read BIDS schema yaml fluently.
Did a few minor fix to pacify pre-commit and make sure the HTML page rendered.
Also check the top message of this PR: https://github.com/bids-standard/bids-specification/pull/1705#issue-2134603705
I added some github admonitions that need tweaking so people know where this BEP is discussed.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 87.92%. Comparing base (
1cb92eb
) to head (73012b3
). Report is 36 commits behind head on master.
:exclamation: Current head 73012b3 differs from pull request most recent head a46e439
Please upload reports for the commit a46e439 to get more accurate results.
Additional details and impacted files
@@ Coverage Diff @@
## master #1705 +/- ##
==========================================
+ Coverage 87.79% 87.92% +0.13%
==========================================
Files 16 16
Lines 1360 1375 +15
==========================================
+ Hits 1194 1209 +15
Misses 166 166
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.