-
Notifications
You must be signed in to change notification settings - Fork 42
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
Document somewhere the special cbrain flag for Boutiques descriptors #1128
Comments
Also code bases (templates) contain I see also
Could go somewhere into developer's wiki, readme of cbrain-neuro-plugins or cbrain.ca. Also I believe @natacha-beck had a wiki on boutiques integration, not sure why it did not get into the main wiki. Update, checked today |
@MontrealSergiy here is the page you are refering: https://github.com/natacha-beck/cbrain-wiki/wiki/Tasks-Integration-With-Boutiques It was document about old Boutiques integration. We should at least have documentation about both way to integrate Boutiques (in cbrain_task_descriptors and in boutiques_descriptors). |
actually there is a designated CBRAIN Boutiques wiki section now, is it a good place for it? |
The current draft documentation is at https://docs.google.com/spreadsheets/d/1GdetgwsxfX9dXDXpqlwi_CU2_Shvn1u1L5yVcZU31g0 |
I want this documentation to be part of the codebase. I am not sure which file is more appropriate, but given we have a library for loading and accessing Boutiques descriptor files, maybe we should put it there. A larger comment block should enumerate them all in a table. Then we could have a nice new set of accessor methods for each of the special flags, something like # Return the CBRAIN author of the descripto from the custom field
# "custom": { "cbrain:author": "Name of person" }
def custom_cbrain_author
self.custom['cbain:author']
end This would also allow us to handle gracefully the inconsistencies we have in these custom sections (e.g. some descriptors have |
Once the methods are all nicely created, we can modify the integration code to use these methods instead of trying to directly access the |
…roperty list to html/txt table aces#1128
The CBRAIN Boutiques integrator supports a set of special flags in the
"custom"
field of the descriptor.Document them all somewhere?
At the top of my head:
The text was updated successfully, but these errors were encountered: