-
Notifications
You must be signed in to change notification settings - Fork 170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BEP022 numbering conflict for Spectroscopy BEP #89
Comments
Oh I didn't saw this BEP. When I was asked to add BIDS Extension Proposal (BEP): Computed tomography scan (CT), I found also that a BIDS Extension Proposal 22(BEP022): PET Preprocessing Derivatives was created so I added it, without checking the existence of others BEP. |
Sounds good to me - I sent an update to the PET Preprocessing Derivatives google doc changing it to 23. Could you send another PR adding MRS under 23 and changing PET derivatives to 22? Thanks! |
I think you mean PR adding MRS under 22 and changing PET derivatives to 23 and Computed Tomography to 24 ? |
Yes
…On Tue, Nov 20, 2018, 8:53 AM Hboni ***@***.*** wrote:
I think you mean PR adding MRS under 22 and changing PET derivatives to 23
and Computed Tomography to 24 ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#89 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAOkp_zWwqai4firJFbnmIRMWtQjr5Seks5uxDOggaJpZM4YpOWl>
.
|
Can be closed ? |
BIDS Extension Proposal 22 (BEP022): Magnetic Resonance Spectroscopy (MRS) was announced awhile back by Dickson Wong (couldn't find github handle). I thought to add it to the registry of existing BEPs within this repo but found that BEP022 was already taken for "PET Preprocessing derivatives". Conflict was already identified by Roeland Hancock and confirmed by @filo but not reported here.
The text was updated successfully, but these errors were encountered: