search

incf-nidash / nidm-specs

Neuroimaging Data Model (NIDM): describing neuroimaging data and provenance
nidm.nidash.org
Other
33 stars 30 forks source link

fix: some parsers don't like a # in @base #417

Closed satra closed 6 years ago

cmaumet commented 7 years ago

Hi @satra! I remember encountering an error message like this in the past but I thought this was caused by a bug in early versions of rdflib (that was then solved). But are you saying here that # are indeed forbidden for @base?

satra commented 7 years ago

@cmaumet - according to the spec it doesn't really seem so. i just checked again and indeed it is the same as prefix. but several things have raised a warning with the # in there (i wish the parsers would fix it). the other thing is that we don't use the base anywhere in that document. so perhaps we can just remove it for now.

cmaumet commented 7 years ago

+1 to remove base as suggested. Let's add this to the next minor release of NIDM-Results?

satra commented 6 years ago

i'm going to close this PR as this is not the solution. we can remove @base as the shorter term solution is implicit base is not being used.