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

New DisGeNET data retrieval and additional features/adjustments (such as HPO retrieval using ontology). #2

Merged
merged 64 commits into from
May 7, 2018

Conversation

svandenhoek
Copy link
Collaborator

Rewritten the DisGeNET data retrieval, added phenotype retrieval using HPO ontology, further implemented output file writing, some adjustments for better looking code, adjustments/added unit-tests.

…s specific URI based on HPO/DisGeNET dataset). Allows for reduced number of database requests if HPO parent/child retrieval is separated from DisGeNET database.
…Entity creation (prefix not optional anymore).
…ith a collection of phenotypes connected to it).
…rk, bug fix regarding same Phenotype being crossed with different distances, generalisation input phenotypes requirement).
…fic retrieval from a collection of PhenotypeNetworks.
…one both inside a class and before adding to the respective class).
…hen adding Phenotypes was removed to allow for more algorithm flexibility when adding new Phenotypes. Due to this, internal List was replaced by a Map.
…her maxDistance even if the same number of phenotypes were returned (now validates for both determining which phenotypes to visit next and for adding to the network instead of only the latter).
…s still need to implement a static method to support this, though can also be used outside enums themselves as long as enum implemented the interface).
…w optional using different RunMode, et cetera).
…parators (also added a convenience method for file closing).
@joerivandervelde joerivandervelde merged commit 017ac9e into molgenis:master May 7, 2018
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