search

PlantPhenoOntology / ppo

An ontology for describing the phenology of individual plants and populations of plants, and for integrating plant phenological data across sources and scales.
16 stars 8 forks source link

bco_ppo_import_module.owl points to bad URL For bco #20

Closed jdeck88 closed 7 years ago

jdeck88 commented 7 years ago

The source is pointing to "http://purl.obolibrary.org/obo/bco/releases/2015-04-10/bco.owl" which gives a 404 error.

It should actually be: http://purl.obolibrary.org/obo/bco/bco.owl

which redirect properly to:

https://raw.githubusercontent.com/BiodiversityOntologies/bco/master/releases/2015-04-10/bco.owl

stuckyb commented 7 years ago

That URL is coming directly from the versionIRI tag in the BCO. If it is invalid, it needs to be updated in the BCO. Until then, the import module still is working fine for me, even if the source IRI is invalid.

stuckyb commented 7 years ago

@jdeck88, is this still an issue?

jdeck88 commented 7 years ago

I believe it is... however, i'm not 100% sure. I'm going to dive into code next week again... been out for holidays and family last couple of weeks so haven't looked at this in awhile.

ramonawalls commented 7 years ago

http://purl.obolibrary.org/obo/bco/bco.owl now redirects to https://raw.githubusercontent.com/tucotuco/bco/master/bco.owl, so it should be pulling the correct file. However, the bco module still includes http://purl.obolibrary.org/obo/bco/releases/2015-04-10/bco.owl as the source. This is strange, because https://github.com/PlantPhenoOntology/PPO/blob/master/src/imports/imported_ontologies.csv is pointing to the PURL for BCO. Is it possible, @stuckyb, that you have not updated the BCO import module since early December? If so, that would explain the inconsistency.

stuckyb commented 7 years ago

Yes, indeed! I've updated the BCO module and it now points to the latest release. @jdeck, does this resolve the issue?

jdeck88 commented 7 years ago

yes, that resolves the issue!