vistalab / PRFmodel

MIT License
11 stars 7 forks source link

extra-bids requirements: sessions? #74

Open Remi-Gau opened 2 years ago

Remi-Gau commented 2 years ago

any reason why the BIDS session entity is REQUIRED in most "APIs"?

this seems like an extra constraint that would force some people extra data wrangling where as there are tools like pybids and or bids-matlab that can be used to query BIDS dataset to help create more flexible APIs.

garikoitz commented 2 years ago

Agreed Remi, in the first version we did a minimally BIDS compatible version. Right now we are working on a more comprehensive version, not for the synthetic datasets (which was the original use), but for the experimental datasets. We are creating another container called prfprepare. and we are editing prfanalyze to be compatible. Can you wait a couple of weeks? If you want to contribute we can have a small meeting and decide together (you have one opportunity this Wednesday at 11:00am Central European time, for example, please let me know) Thanks again for your interest! Gari

On Mon, Mar 14, 2022 at 3:38 PM Remi Gau @.***> wrote:

any reason why the BIDS session entity is REQUIRED in most "APIs"?

this seems like an extra constraint that would force some people extra data wrangling where as there are tools like pybids and or bids-matlab that can be used to query BIDS dataset to help create more flexible APIs.

— Reply to this email directly, view it on GitHub https://github.com/vistalab/PRFmodel/issues/74, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCZAVZT2EUKA5F57EM3IM3U75FOLANCNFSM5QVXR4SQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Remi-Gau commented 2 years ago

hey @garikoitz

thanks for the quick response

this can totally wait on my side: this was a more of a general comment on my side as a BIDS maintainer

won't be able to join on Wednesday

but I could join later to give you my 2 cents:

garikoitz commented 2 years ago

I see, thanks! we will look into it

On Tue, Mar 15, 2022 at 1:15 PM Remi Gau @.***> wrote:

hey @garikoitz https://github.com/garikoitz

thanks for the quick response

this can totally wait on my side: this was a more of a general comment on my side as a BIDS maintainer

won't be able to join on Wednesday

but I could join later to give you my 2 cents:

— Reply to this email directly, view it on GitHub https://github.com/vistalab/PRFmodel/issues/74#issuecomment-1067917746, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCZAV4AEJE75JY3XCIZOK3VAB5NFANCNFSM5QVXR4SQ . You are receiving this because you were mentioned.Message ID: @.***>

Remi-Gau commented 2 years ago

Other things that exist in the BIDS world that you could align with:

Remi-Gau commented 2 years ago

oh and of course those are just friendly suggestions: I have 0 idea how far down the road you are to those new features and how much flexibility you have regarding API definition.