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

revise vaccine.csv #729

Merged
merged 3 commits into from
Jul 31, 2024
Merged

revise vaccine.csv #729

merged 3 commits into from
Jul 31, 2024

Conversation

xingxianli
Copy link
Contributor

updated equivalent class in vaccine.csv, revised term "bacterial vaccine"/"viral vaccine" to "vaccine", then some vaccine immunizes against both bacteria and virus can be correctly inferred

Copy link
Contributor

@zhengj2007 zhengj2007 left a comment

Choose a reason for hiding this comment

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

Could also update few other high-level vaccine terms same as those bacterial and viral vaccines?

  1. bacterial vaccine
  2. fungal vaccine
  3. protozoan vaccine
  4. viral vaccine
  5. cancer vaccine
  6. all direct subClasses of 'vaccine by platform type'

In the equivalent axiom, change specific vaccine type to 'vaccine'.

Besides, could you please relabel 'vaccine for pathogen' to 'vaccine against pathogen' and 'vaccine for disease' to 'vaccine against disease'?
And add 'term editor': Barry Smith, Oliver He, Jie Zheng, Charles Tapley Hoyt, Xingxian Li; Add 'term tracker item' #619

In addition, two subClasses of 'varicella zoster virus glycoprotein E' are vaccine, need to change as what we have done for vaccine antigen terms. (It needs to add new term, 'varicella zoster virus glycoprotein E vaccine')

Please let me know if you have any questions.

@zhengj2007 zhengj2007 self-requested a review July 31, 2024 17:49
@zhengj2007 zhengj2007 merged commit 9b04f4a into master Jul 31, 2024
@zhengj2007 zhengj2007 deleted the vaccine branch July 31, 2024 20:09
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