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

Fix tsv formatting and refresh imports #238

Merged
merged 10 commits into from
Apr 28, 2023
Merged

Fix tsv formatting and refresh imports #238

merged 10 commits into from
Apr 28, 2023

Conversation

rays22
Copy link
Contributor

@rays22 rays22 commented Apr 26, 2023

  1. Replace space characters with tabs in entity_attribute_location.tsv. Remove OBA:2060177, because it is a duplicate of OBA:VT0000203. Removing OBA:2060177 also solves the tsv formatting issue.
  2. Refresh imports
    • Fix the consequence of refreshing the GO import that brings in a newly obsoleted process term used by OBA

Replace space characters with tabs in entity_attribute_location.tsv.
@rays22 rays22 self-assigned this Apr 26, 2023
rays22 added 3 commits April 27, 2023 10:47
This commit intends to provide a short term fix for the issues caused by the obsoletion of the process term [aging GO:0007568](geneontology/go-announcements#502).
Need to add back these terms once there are appropriate replacement terms for [GO:0007568](https://www.ebi.ac.uk/ols/ontologies/GO/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FGO_0007568). See related issue #239.
entity_attribute_location.tsv
198:OBA:VT0000203       blood aspartate transaminase amount     PR:000008153    aspartate aminotransferase, cytoplasmic PATO:0000070    amount  UBERON:0000178  blood
251:OBA:2060177         PR:000008153    aspartate aminotransferase, cytoplasmic PATO:0000070    amount  UBERON:0000178  blood
@rays22 rays22 changed the title Fix tsv formatting Fix tsv formatting and refresh imports Apr 27, 2023
@rays22 rays22 requested review from ar-ibrahim and matentzn April 27, 2023 15:51
@rays22 rays22 requested a review from matentzn April 28, 2023 11:05
The process of `aging GO:0007568` [has been obsoleted in GO.](geneontology/go-announcements#502).
The affected OBA trait terms

```
OBA:0003870
OBA:0005494
```

have been added back without EQs manually.
id: OBA:0003870
name: aging onset quality
def: "The onset quality of aging." []
is_a: PATO:0002325 ! onset quality
Copy link
Contributor

Choose a reason for hiding this comment

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

parent must be an OBA term, else violates constraints

id: OBA:0005494
name: aging rate
def: "The rate of the aging process." []
is_a: PATO:0000161 ! rate
Copy link
Contributor

Choose a reason for hiding this comment

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

parent must be an OBA term

Copy link
Contributor

Choose a reason for hiding this comment

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

Diff still large

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The large diff is caused by missing tab characters between empty cells in the tsv table in the last two columns in the master version of the tsv file. I am not sure if that is correct syntax for a tsv file, but somebody's text editor thought so. Anyway, I have decreased the diff by removing the tab characters from my version of the file.

@rays22 rays22 merged commit eaca255 into master Apr 28, 2023
@rays22 rays22 deleted the fix_tsv2023-04-26 branch April 28, 2023 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants