Skip to content
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

Move config functions into roar-utils #10

Open
richford opened this issue Jan 25, 2023 · 4 comments
Open

Move config functions into roar-utils #10

richford opened this issue Jan 25, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@richford
Copy link
Contributor

The functions imported here should be included in roar-utils. Additionally, this function should also include some data standardization.

@richford richford added the enhancement New feature or request label Jan 25, 2023
@richford
Copy link
Contributor Author

For data hygiene, the trial data should conform to:

  • If there is a correct field, it should be converted to boolean
  • Field names blockNum and blockId should be prohibited. Prefer block instead.
  • theta and thetaSE should be numbers. thetaSE should be non-negative.

@richford
Copy link
Contributor Author

The three functions initConfig, initRoarJsPsych, and initRoarTimeline should probably be converted into a class that gets initialized and then has the four props config, jsPsych, timeline, and firekit.

@Mohammed-Shoaib
Copy link
Collaborator

The class should take taskInfo as a property of the class.

@Mohammed-Shoaib
Copy link
Collaborator

c = 1, completion token
c = 0, completed but not finished (not sure)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants