bids-specification icon indicating copy to clipboard operation
bids-specification copied to clipboard

[ENH] microelectrode electrophysiology specification (BEP032)

Open yarikoptic opened this issue 1 year ago • 5 comments

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]

HTML preview of this BEP


  • 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 the Details link of the ci/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

yarikoptic avatar Feb 14 '24 15:02 yarikoptic

Also relevant if you'd like to comment. → https://docs.google.com/document/d/1oG-C8T-dWPqfVzL2W8HO3elWK8NIh2cOCPssRGv23n0/edit?disco=AAABIzHGpUU

TheChymera avatar Mar 12 '24 01:03 TheChymera

Also I forgot to link to this when I wrote it → https://docs.google.com/document/d/1oG-C8T-dWPqfVzL2W8HO3elWK8NIh2cOCPssRGv23n0/edit?disco=AAABIGPAMOw

TheChymera avatar Mar 12 '24 01:03 TheChymera

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.

Remi-Gau avatar Apr 18 '24 07:04 Remi-Gau

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.

Remi-Gau avatar Apr 19 '24 07:04 Remi-Gau

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.

codecov[bot] avatar Apr 19 '24 22:04 codecov[bot]