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.

Sign up for GitHub

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

Jump to bottom

Update biolink-model.yaml for CTKP requirements #1526

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update biolink-model.yaml for CTKP requirements #1526

wants to merge 3 commits into from

Conversation

mbrush
Copy link
Collaborator

@mbrush mbrush commented Oct 7, 2024

  • Added 'has supporting studies' edge property
  • Added comments to 'has supporting study metadata' edge property to clarify when these properties should be used
  • Removed 'Confidence Level' as range of 'has confidence level'
  • Changed the name of the MaxResearchPhaseEnum to just ResearchPhaseEnum - so we can use values to represent the phase of a particular trial (and updated reference to this enum form the 'max research phase' edge property accordingly)

@mbrush
Update biolink-model.yaml
416649e 
- Added 'has supporting studies' edge property
- Added comments to 'has supporting study metadata' edge property to clarify when these properties should be used
- Removed 'Confidence Level' as range of 'has confidence level'
- Changed the name of the MaxResearchPhaseEnum to just ResearchPhaseEnum - so we can use values to represent the phase of a particular trial (and updated reference to this enum form the 'max research phase' edge property accordingly)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sierra-moxon sierra-moxon sierra-moxon approved these changes

@gglusman gglusman Awaiting requested review from gglusman

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mbrush @sierra-moxon