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 Brochu 2003 to the ontology of phyloreferences #29

Merged
merged 22 commits into from
Jun 18, 2018

Conversation

gaurav
Copy link
Member

@gaurav gaurav commented Apr 14, 2018

This pull request adds the publication Brochu 2003 as curated by the Curation Tool and incorporates it into the Ontology of Phyloreferences.

Bugs that have been fixed in this pull request are:

It also incorporates JPhyloref improvements (phyloref/jphyloref#6, phyloref/jphyloref#9) that implements statuses for phyloreferences (phyloref/klados#25).

Some revealed have been filed to work on later, and are not resolved in this pull request:

This has now been tested in Travis and is ready for review and to be merged.

@gaurav gaurav requested review from hlapp and removed request for hlapp April 30, 2018 22:30
@hlapp
Copy link
Member

hlapp commented May 14, 2018

@gaurav can you say what's left or missing here? I've lost track and it's now two weeks old.

@gaurav
Copy link
Member Author

gaurav commented May 22, 2018

Sorry, @hlapp -- I forgot to open an issue for marking phyloreferences that could not be resolved. I've done so now at #31, which is what's holding this pull request open. I'd hoped to finish that by this week but it's taking longer than expected.

Copy link
Member

@hlapp hlapp left a comment

Choose a reason for hiding this comment

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

We should add to the Phyloref Ontology a list of all terms that we are using here from other community ontologies. We should have these all in the Phyloref Ontology, so that code only has to worry about referencing that ontology, and not 8 others.

@gaurav gaurav merged commit 40e3a11 into master Jun 18, 2018
@gaurav gaurav deleted the example_brochu_2003 branch June 18, 2018 20:15
gaurav added a commit to phyloref/jphyloref that referenced this pull request Jun 20, 2018
The current version of the phyloreference testing code had evolved through several different cycles, was badly written, and appeared to not be correctly testing phyloreferences using the expected_phyloreference_named property. I replaced it with cleaner, easier to read code that appears to be testing phyloreferences correctly, and tested these in PR #9 and its use in phyloref/clade-ontology#29.
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