-
Notifications
You must be signed in to change notification settings - Fork 5
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
0.4.5 #85
Conversation
# Conflicts: # phenopacket-tools-validator-core/src/main/java/org/phenopackets/phenopackettools/validator/core/impl/DefaultPhenopacketIngestor.java
Added docs according to Harshad's walkthrough
@ielis @julesjacobsen -- this is working now, I have added a prototype Ontology validation class. There are some things that potentially should be changed -- in particular, it is not straightforward just to pass "Message" (protobuf) to the validator instead of "Phenopacket". Also, we will need to figure out how to use multiple JSON Schema files and link the Phenopacket to Family and Cohort. |
Add Sphinx documentation
…wRunner`. Signed-off-by: Daniel Danis <[email protected]>
…wRunner`. Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
…ts, families, and cohorts. Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
…ing of absence of the main `Family` components. Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
…r hierarchy of the proto files in `phenopacket-schema`. Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
Signed-off-by: Daniel Danis <[email protected]>
… and `main`. Signed-off-by: Daniel Danis <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
proposed API -- simplified a few things. If we like this general structure, we would need some analogous classes for Cohort and Family messages.