buda-base / owl-schema

BDRC Ontology Schema
11 stars 2 forks source link

admin prefix #17

Closed eroux closed 6 years ago

eroux commented 7 years ago

It seems either Jena or jsonld-java (I suspect the latter but I need to inspect further) doesn't like the root prefix being included in the admin prefix, I'm getting weird output like

{
  "@graph" : [ {
    "admin/place_town_ch" : "蔡公堂乡",
  } ],
  "@context" : {
    "@vocab" : "http://purl.bdrc.io/ontology/",
    "" : "http://purl.bdrc.io/ontology/",
    "adm" : ":admin/",
    "bdd" : "http://purl.bdrc.io/data/",
    "bdr" : "http://purl.bdrc.io/resource/",
  }
}
eroux commented 7 years ago

fixed (was a config problem, not really about the schema)

eroux commented 7 years ago

https://github.com/jsonld-java/jsonld-java/issues/200

eroux commented 6 years ago

I really thought the thing was looking ugly, so I tried to fix jsonld-java, but the problem is deeper: https://github.com/json-ld/json-ld.org/issues/526 so maybe we could change the prefixes and add a core/ for the main thing? wdyt?

xristy commented 6 years ago

why not just leave bdo alone and change admin to:

xmlns:admin="http://purl.bdrc.io/admin/"

eroux commented 6 years ago

why not yes ok, we need to move tbr too