search

individual-brain-charting / public_analysis_code

Public analysis code for the IBC project
Other
20 stars 7 forks source link

Naming issue with FingerTap (FBIRN) #34

Closed ferponcem closed 1 year ago

ferponcem commented 1 year ago

In all_contrasts the finger tapping tasks is called FingerTapping, which I think it's a mistake, because in the files the name is res_task-FingerTap_space... ; and same in utils_contrast. @bthirion So if there's no other place where this could be a problem, I could directly correct this in the all_contrasts.csv file.

bthirion commented 1 year ago

+1 for consistency, so lets' move to FingerTap for the moment... but we'll probably move later all occurrences to FingerTapping later,as FingerTap is obviously not a good name.

ferponcem commented 1 year ago

@bthirion The thing is that we already have a FingerTapping: FingerTapping -> CamCAN FingerTap -> FBIRN

bthirion commented 1 year ago

OK, makes sense. Then let's go for FingerTap :-(

ferponcem commented 1 year ago

Thanks!