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

Fixes #2169 relabel lung cell types #2403

Merged
merged 7 commits into from
Jun 28, 2024
Merged

Conversation

aleixpuigb
Copy link
Collaborator

Fixes #2169 relabel lung cell types

@aleixpuigb aleixpuigb added text definition relabel HuBMAP Needed/useful for HuBMAP labels Jun 20, 2024
@aleixpuigb aleixpuigb self-assigned this Jun 20, 2024
@aleixpuigb
Copy link
Collaborator Author

#gogoeditdiff

@aleixpuigb aleixpuigb marked this pull request as draft June 20, 2024 15:12
Copy link

Here's a diff of how these changes impact the classified ontology (on -simple file):

Ontology comparison

Left

  • Ontology IRI: http://purl.obolibrary.org/obo/cl/cl-simple.owl
  • Version IRI: http://purl.obolibrary.org/obo/cl/releases/2024-06-20/cl-simple.owl
  • Loaded from: file:/__w/cell-ontology/cell-ontology/src/ontology/cl-simple-master.owl/cl-simple.owl

Right

  • Ontology IRI: http://purl.obolibrary.org/obo/cl/cl-simple.owl
  • Version IRI: http://purl.obolibrary.org/obo/cl/releases/2024-06-20/cl-simple.owl
  • Loaded from: file:/__w/cell-ontology/cell-ontology/src/ontology/cl-simple-pr.owl/cl-simple.owl

Ontology imports

Ontology annotations

alveolar type 2 fibroblast cell http://purl.obolibrary.org/obo/CL_4028006

Removed

Added

pneumocyte http://purl.obolibrary.org/obo/CL_0000322

Removed

Added

type I pneumocyte http://purl.obolibrary.org/obo/CL_0002062

Removed

Added

type II pneumocyte http://purl.obolibrary.org/obo/CL_0002063

Removed

Added

Copy link

Here's a diff of your edit file (unreasoned)

Ontology comparison

Left

  • Ontology IRI: http://purl.obolibrary.org/obo/cl.owl
  • Version IRI: None
  • Loaded from: file:/__w/cell-ontology/cell-ontology/master/src/ontology/cl-edit.owl

Right

  • Ontology IRI: http://purl.obolibrary.org/obo/cl.owl
  • Version IRI: None
  • Loaded from: file:/__w/cell-ontology/cell-ontology/branch/src/ontology/cl-edit.owl

Ontology imports

Ontology annotations

alveolar type 2 fibroblast cell http://purl.obolibrary.org/obo/CL_4028006

Removed

Added

pneumocyte http://purl.obolibrary.org/obo/CL_0000322

Removed

Added

type I pneumocyte http://purl.obolibrary.org/obo/CL_0002062

Removed

Added

type II pneumocyte http://purl.obolibrary.org/obo/CL_0002063

Removed

Added

@aleixpuigb
Copy link
Collaborator Author

A new term 'adventitial fibroblast' is needed.

@aleixpuigb aleixpuigb marked this pull request as ready for review June 21, 2024 10:25
@aleixpuigb aleixpuigb requested a review from JABelfiore June 21, 2024 10:25
Copy link
Contributor

@dosumis dosumis left a comment

Choose a reason for hiding this comment

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

See comment on def AT2. (sorry I didn't spot this first time)

src/ontology/cl-edit.owl Show resolved Hide resolved
@aleixpuigb aleixpuigb requested a review from dosumis June 24, 2024 11:02
@aleixpuigb
Copy link
Collaborator Author

Thanks for the review @dosumis. I have done the edits.

@aleixpuigb aleixpuigb requested review from AvolaAmg and removed request for dosumis and JABelfiore June 25, 2024 12:30

AnnotationAssertion(Annotation(oboInOwl:hasDbXref "GOC:tfm") Annotation(oboInOwl:hasDbXref "PMID:20054144") Annotation(oboInOwl:hasDbXref "http://www.copewithcytokines.de") obo:IAO_0000115 obo:CL_0002062 "A type I pneumocyte is a flattened, branched pneumocyte that covers more than 98% of the alveolar surface. This large cell has thin (50-100 nm) cytoplasmic extensions to form the air-blood barrier essential for normal gas exchange.")
AnnotationAssertion(Annotation(oboInOwl:hasDbXref "GOC:tfm") Annotation(oboInOwl:hasDbXref "PMID:20054144") Annotation(oboInOwl:hasDbXref "PMID:36138472") Annotation(oboInOwl:hasDbXref "http://www.copewithcytokines.de") obo:IAO_0000115 obo:CL_0002062 "A squamous pulmonary alveolar epithelial cell that is flattened and branched. A pulmonary alveolar type 1 cell covers more than 98% of the alveolar surface. This large cell has thin (50-100 nm) cytoplasmic extensions to form the air-blood barrier essential for normal gas exchange.")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi there, this paper PMID:36138472 refers to it as pulmonary alveolar type 2 cell rather than pulmonary alveolar type1 cell. Could you use a different reference maybe this one PMID: 29463737 ?

Copy link
Collaborator Author

@aleixpuigb aleixpuigb Jun 27, 2024

Choose a reason for hiding this comment

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

Hi @AvolaAmg , I will update the reference, thank you. It is true that while mentioning that pulmonary alveolar type 1 are squamous cells, it is not the main focus on the paper and your proposed is more adequate. Cheers!

@aleixpuigb aleixpuigb dismissed dosumis’s stale review June 28, 2024 08:33

Reviewer is not available and review comments have been adressed.

@aleixpuigb aleixpuigb merged commit 9b12dff into master Jun 28, 2024
1 check passed
@aleixpuigb aleixpuigb deleted the 2169_relabel_lung_cell_types branch June 28, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update labels to pulmonary terms
3 participants