Skip to content

Conversation

Peyman-N
Copy link
Member

@Peyman-N Peyman-N commented Mar 4, 2024

No description provided.

@Peyman-N Peyman-N marked this pull request as ready for review August 5, 2024 12:37
Copy link
Member

@lzehl lzehl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TBD : current state does not fit yet

"type": "array",
"uniqueItems": true
},
"behavioralProtocol": {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this does not make sense here. please remove

"assessment": {
"_instruction": "Add All the assessments aquired during this session.",
"_linkedTypes": [
"https://openminds.ebrains.eu/specimenPrep/Assessment"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TBD: I do not think this is a separate schema

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

most likely this will be a behavioral protocol

@@ -0,0 +1,21 @@
{
"_extends": "/publications/schemas/creativeWork.schema.tpl.json",
"_type": "https://openminds.ebrains.eu/specimenPrep/AssessmentInstrument",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

type mismatches type of filename and of reference in the other schema. Plus: I don't think this is a separate schema. Questionnaires or behavioral assessments are captured in behavioral protocols.

@@ -0,0 +1,36 @@
{
"_extends": "/core/schemas/research/experimentalActivity.schema.tpl.json",
"_type": "https://openminds.ebrains.eu/specimenPrep/SubjectPreparation",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

name is too generic.

@lzehl
Copy link
Member

lzehl commented Sep 24, 2024

Note: this PR should be updated after the merge of #15

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

Successfully merging this pull request may close these issues.

2 participants