-
Notifications
You must be signed in to change notification settings - Fork 54
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
[ENH] publish notes of the TownHall #459
Conversation
Remi-Gau
commented
Jul 26, 2024
•
edited
Loading
edited
- closes publish slides and notes of the BIDS townhall #455
- update BEP status according to what was in the slides
- 'Persperctive: Start merging of stand-alone parts of the BEP (i.e., neuronavigation)' | ||
- 'Persperctive: Collaboration with common NIBS software (BEST toolbox, simNIBS, K-space…)' | ||
- 'Persperctive: Apply for funds for pushing forward the BEP' | ||
- Updated BEP structure that eliminates new `.json` or `.tsv` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update on NIBS BEP
os.environ["OPEN_CITATIONS_ACCESS_TOKEN"] = "/home/remi/Documents/tokens/open_citations_access_token.txt" | ||
os.environ["OPEN_CITATIONS_ACCESS_TOKEN"] = ( | ||
"/home/remi/Documents/tokens/open_citations_access_token.txt" | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how big is the token -- should we place it into secrets and automate this on CI?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in any case -- IMHO better to use some env var and not hardcode paths (or evil yarik would login and steal that exact file!)
Co-authored-by: Yaroslav Halchenko <[email protected]>