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

Add publication 1 0 0 #11

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add publication 1 0 0 #11

wants to merge 4 commits into from

Conversation

ishitasati11
Copy link
Collaborator

Added Publication file

properties:
properties:
project_id:
description: Primary key or unique identified for a project
Copy link
Collaborator

Choose a reason for hiding this comment

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

@esheetah typo - identifier*

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Correcting this

description: Link to the ontology file
url: https://bioportal.bioontology.org/ontologies/ORTH?p=classes&conceptid=http%3A%2F%2Fpurl.org%2Fnet%2Forth%23Organism
cell_line:
description: Identifier of the cell lines from which the samples were derived for this experiment
Copy link
Collaborator

Choose a reason for hiding this comment

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

@esheetah The description here should simply be the answer to "What is a cell-line"? Description is not a metadata field for the physical data model. It is simply to add more information about the attributes as a part of the conceptual model definition.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Example section will cover what kind of values can be stored along with the above definition which fits better in the example section.

type: float
format: numeric
example: 70
upper_value: 10000.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

@esheetah Use upper limit and lower limit

@@ -0,0 +1,437 @@
Protocol:
Copy link
Collaborator

Choose a reason for hiding this comment

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

@esheetah A question here. Is this definition specific to a data type? I'd like to ask this because the definition needs to be generic to apply to all data types. If not, we must create a base/core definition followed by data type-specific modules.

authors:
description: List of authors for the associated publication
type: string
example: "Puyalto A, Rodríguez-Remírez M, López I, Macaya I"
Copy link
Collaborator

Choose a reason for hiding this comment

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

@esheetah Publications follow APA style formatting in most cases. This example is APA. Let's add that in the description somewhere?
Also, the initials of the first name will end with a . e.g. Puyalto, A.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Adding it in the description

type: string
description: Link to the ontology file
url: https://bioportal.bioontology.org/ontologies/OBI?p=classes&conceptid=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FOBI_0001617
total_number_of_cells:
Copy link
Collaborator

Choose a reason for hiding this comment

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

@esheetah Looks like a data-type specific field. Need to segment these better

@@ -0,0 +1,88 @@
Publication:
Copy link
Collaborator

@pverma12394 pverma12394 May 29, 2024

Choose a reason for hiding this comment

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

@esheetah Use of double quotes in the example section is inconsistent. Let's either use it or not use it at all. Please review this in other entities wherever applicable

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The examples with string values are given in double quotes, the integer/float examples are not. Should we be giving the numerical values in double quotes as well? @pverma12394

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.

None yet

3 participants