-
Notifications
You must be signed in to change notification settings - Fork 9
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
[DOC] docs update part 1 #123
[DOC] docs update part 1 #123
Conversation
Codecov Report
@@ Coverage Diff @@
## doc #123 +/- ##
======================================
Coverage ? 81.00%
======================================
Files ? 29
Lines ? 616
Branches ? 0
======================================
Hits ? 499
Misses ? 117
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@@ -2,29 +2,18 @@ | |||
|
|||
function createDataDictionary(cfg, logFile) | |||
% | |||
% Short description of what the function does goes here. | |||
% It creates the data dictionary to be associated with a _events.tsv file. It will create empty |
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.
I think that in the long term this function should be able to write the dictionary for a wider variety of data files and not just events file. At least stim files too. Hopefully. 😉
@@ -2,29 +2,18 @@ | |||
|
|||
function createDataDictionary(cfg, logFile) | |||
% | |||
% Short description of what the function does goes here. | |||
% It creates the data dictionary to be associated with a _events.tsv file. It will create empty |
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.
Also the function can use the content of cfg
to create those data dictionary so it will only create empty fields if the user has not set them before.
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
% | ||
% The behavior of this function depends on: | ||
% - ``cfg.testingDevice``: | ||
% - set to ``pc`` (dummy try) or ``beh`` can work for behavioral experiment. |
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.
I think at the moment we only accept pc
but it would make sense to have beh
as well I think. Feel free to open an issue so we remember to do that.
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
Co-authored-by: Remi Gau <remi_gau@hotmail.com>
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.
Good to go: 🚀 🎉 🧙♂️
No description provided.