Update wavelengthIndex description for processed data#175
Update wavelengthIndex description for processed data#175sreekanthkura7 wants to merge 2 commits into
Conversation
Clarified requirements for wavelengthIndex in processed data.
HanBnrd
left a comment
There was a problem hiding this comment.
Thanks @sreekanthkura7, good catch! Looks good to me!
|
I guess this change would require a respective adjustment in the validator, right? Is @sstucker still the designated validator maintainer? |
|
is this the final convention that has been decided on? Some readers have made their own convention, e.g. Cedalion uses -1 https://github.com/ibs-lab/cedalion/blob/main/src/cedalion/io/snirf.py#L1003 |
The validator has a bit of backlog on the specs, we could sort this as part of #183
@sreekanthkura7 could we do |
Clarified requirements for wavelengthIndex in processed data.
Closes #174