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

Added new protein term #291

Merged
merged 10 commits into from
Sep 15, 2023
Merged

Added new protein term #291

merged 10 commits into from
Sep 15, 2023

Conversation

ar-ibrahim
Copy link
Collaborator

@ar-ibrahim ar-ibrahim self-assigned this Sep 11, 2023
@ar-ibrahim ar-ibrahim marked this pull request as ready for review September 11, 2023 15:21
Copy link
Contributor

@rays22 rays22 left a comment

Choose a reason for hiding this comment

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

The tab characters at the end of lines are missing for the last columns in the table. Please try to edit the tsv file in gsheets so that the tab characters do not get removed.

OBA:2050067 blood cadmium amount CHEBI:22977 cadmium atom PATO:0000070 amount UBERON:0000178 blood
OBA:2050068 serum albumin amount PR:000003918 albumin PATO:0000070 amount UBERON:0001977 serum
OBA:2050070 serum IgM amount GO:0071754 IgM immunoglobulin complex, circulating PATO:0000070 amount UBERON:0001977 serum
OBA:0000016 angiotensin levels in blood PR:000036009 angiotensin II PATO:0000070 amount UBERON:0000178 blood
Copy link
Contributor

Choose a reason for hiding this comment

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

The tab separators for the last columns are missing here and at many similar places.

Copy link
Contributor

Choose a reason for hiding this comment

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

There are still tab characters missing here and elsewhere in the tsv file.

@ar-ibrahim ar-ibrahim marked this pull request as draft September 12, 2023 08:43
ar-ibrahim and others added 2 commits September 12, 2023 10:09
- edited in different text editor (VS Code) to fix tab characters issue

- added additional synonym
@ar-ibrahim ar-ibrahim marked this pull request as ready for review September 12, 2023 09:24
Copy link
Contributor

@rays22 rays22 left a comment

Choose a reason for hiding this comment

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

You need to fix the missing tab characters in the file entity_attribute_location.tsv.
I suggest you follow the instructions I sent you on slack last week.
I am copying and pasting the text here for your convenience.

  1. Save your new line somewhere outside of github control:
OBA:2050349	level of interleukin-12 subunit beta in blood	PR:000001369	interleukin-12 subunit beta	PATO:0000070	amount	UBERON:0000178	blood
  1. Replace the entity_attribute_location.tsv file in your branch with a fresh copy from the master branch (which should have the correct tab characters).
  2. Import/open the file entity_attribute_location.tsv in google sheets (or similar). Add back your new term:
OBA:2050349	level of interleukin-12 subunit beta in blood	PR:000001369	interleukin-12 subunit beta	PATO:0000070	amount	UBERON:0000178	blood
  1. Export the updated table as tsv and replace the entity_attribute_location.tsv file with your edited copy. Check the git diff and the tsv file should show changes at the last line(s) that you added and the tab characters should be retained for empty columns.

OBA:2050067 blood cadmium amount CHEBI:22977 cadmium atom PATO:0000070 amount UBERON:0000178 blood
OBA:2050068 serum albumin amount PR:000003918 albumin PATO:0000070 amount UBERON:0001977 serum
OBA:2050070 serum IgM amount GO:0071754 IgM immunoglobulin complex, circulating PATO:0000070 amount UBERON:0001977 serum
OBA:0000016 angiotensin levels in blood PR:000036009 angiotensin II PATO:0000070 amount UBERON:0000178 blood
Copy link
Contributor

Choose a reason for hiding this comment

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

There are still tab characters missing here and elsewhere in the tsv file.

@ar-ibrahim ar-ibrahim marked this pull request as draft September 12, 2023 12:26
Copy link
Contributor

@rays22 rays22 left a comment

Choose a reason for hiding this comment

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

Looks fine!

@ar-ibrahim ar-ibrahim merged commit 3e5feaa into master Sep 15, 2023
@ar-ibrahim ar-ibrahim deleted the new_protein_term branch September 15, 2023 14:39
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