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

chore: update ontology decendant mappings #116

Conversation

github-actions[bot]
Copy link
Contributor

Automated changes by create-pull-request GitHub action

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@Bento007 Bento007 closed this Mar 15, 2024
Bento007 added a commit that referenced this pull request Mar 15, 2024
## Reason for Change

- #15 

## Changes

- add script to generate descendant mapping. It's based off a script in
[single-cell-curation](https://github.com/chanzuckerberg/single-cell-curation/blob/main/scripts/compute_mappings/compute_tissue_and_cell_type_mappings.py)
- update GHA to install the local API to run generate descendant
mapping.

## Testing steps

- Added test cases
- Compared to tissue_descendants.json and cell_type_descendant.json
generating by the single-cell-curation repo. Results:
- verified PR is auto generated for decendant mapping updates
#116
- Add GHA run every monday at midnight, by workflow_dispatch, or if the
ontology assets are updated

```
------Comparing Ontology Guide and single-cell-curation Cell Type Descendant
KEYS: In Ontology Guide not in single-cell-curation
	 {'CL:4030031'}
DESCENDANT: In Ontology Guide  not in single-cell-curation
	 CL:0000738 {'CL:2000054'}
	 CL:0002320 {'CL:0002554'}
	 CL:0002319 {'CL:4033051', 'CL:4033052'}
	 CL:0000117 {'CL:4033051', 'CL:4033052'}
	 CL:0000057 {'CL:0002554'}
	 CL:0000540 {'CL:4033051', 'CL:4033052'}
	 CL:0000219 {'CL:2000054'}
	 CL:0000542 {'CL:2000054'}
	 CL:0000988 {'CL:2000054'}

KEYS: In single-cell-curation not in Ontology Guide
	 None
DESCENDANT: In single-cell-curation not in Ontology Guide
```
## Notes for Reviewer
- Terms in the locally generated cell_type_descendants.json not in
cellxgene_schema/cell_type_descendants.json
- `CL:4030031`: [interstitial
cell](https://www.ebi.ac.uk/ols4/ontologies/cl/classes/http%253A%252F%252Fpurl.obolibrary.org%252Fobo%252FCL_4030031)
- `CL:0002494`:
[cardiocyte](https://www.ebi.ac.uk/ols4/ontologies/cl/classes/http%253A%252F%252Fpurl.obolibrary.org%252Fobo%252FCL_0002494)
was recently added to
chanzuckerberg/single-cell-data-portal#6756 and
chanzuckerberg/single-cell-curation#779. Not
sure why it isn't showing up in the old version of the descendant
mapping script.

---------

Co-authored-by: github-actions <[email protected]>
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.

1 participant