-
Notifications
You must be signed in to change notification settings - Fork 8
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
Dev #326
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Can run from the minerva command line as it sits. Most likely not useful code once we are done here. Could run it off a branch or merge it in and consider removing it later. or just leave it in.. not harmful. example from minerva-cli --update-gene-product-types -i /Users/bgood/Documents/GitHub/noctua-models/models/ -o /Users/bgood/test/typed_master/ -n /Users/bgood/gocam_ontology/neo_full_nov20_2019.owl -c /Users/bgood/gocam_ontology/catalog-no-import.xml
rather than always downloading from the repo.
Types added to gene nodes will now work for the reasoner and validation but will not be saved in the models. Note that this increases the overall dependency on the presence of GOLR as this is where the types come from.
…asses from classes to expand with golr
travis_mode is used only for go_shapes to fail when something is wrong. This change allows the shex validation to run when the OWL validation succeeds - and when we are expecting a failure. if we want to separate owl and shex validation and use this kind of set up we would want to introduce another yet another parameter to indicate travis-owl travis-shex travis-both ... currently equates to travis-shex
This will eliminate the addition and saving of GP types while still using them during OWL inference and shex validation. (they are gathered dynamically from golr, added to temporary models, and then discarded).
…pes-for-problem-nodes adding information on the shapes that do match troublesome nodes.
add id to order by for #235
fix ordering for model search
…h-response fix search result for users and groups
currently skipping over all nice error handling...
This is a sparql-based version of closing the shape.
In JSON response, will look the same as other shex violations except will show the intended range shapes for the misused property as owl:Nothing like this: "intended-range-shapes": [ "owl:Nothing" ]
Performance go shapes199
addressing shape inheritance issue
…re-exact not exactly exact match.. but this should work better so lets try it.
think this should fix issue #311
…s-search-taxa changing routes for search service
…location change ontojournal location As requested elsewhere.
fixes problem detected in #313 resulting from lack of inference step applied to go-lego including neo. That found that PR_000000001 was equivalent to CHEBI_36080 . All reference in minerva code now use the chebi uri.
changing all root protein to chebi reference
Somehow.. I think the re-use of the go-cam's IRI as the IRI for the schema ontology confused the system. Following a GPAD export, the go-cam model had nothing in it except for all of the object property assertions from the original imported go-lego ontology. Taking that out stabilized my system locally.
seems jdk.internal.org.jline.utils exists on my java but not on travis. Yay!
#316 (comment) also removing unused method addTaxonToDatabaseWithOWL (which was astonishingly slow compared to the RDF version of the same method that replaced it..)
have tbox in manager when loading model from blazegraph
This ought to go into that config file and not get spread all over.. Someday. Regarding: #316 (comment)
removing /tmp/blazegraph.jnl references
…eo-setup minor update - now assumes pre-merged tbox ontology
…n-report-318 Add title to shex validation report 318
pretty dumb. should work now. (also sneaking in an option to report some reasoning results on the models tested. add --reasoner_report to see).
…n-report-318 fixing title printing to file..
Catch |
People have been testing for a while, ready to do. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
readying master branch for noctua upgrade. Encapsulates many changes.