Allow specifying input type in robot convert
#1038
Comments
|
These parsing messages are a nightmare.. I agree, and I would even like to go a bit further and widdle down the error message in the stack trace a bit. I think this is a good idea @cthoyt and should not be too hard to implement.. I wont do it right now, but if pressure mounts on this ticket I might be convinced to do it. |
|
Another reason to prioritize: junk input such as the html we get back from
a misconfigured purl is frequently valid input for *some* parser, so
parsing silently passes producing an empty ontology when it should fail. A
lot of people have been tripped up by this. If we were designing from
scratch I would force specifying an input type and only iterate through all
parsers if explicitly requested
…On Fri, Aug 5, 2022 at 3:10 PM Nico Matentzoglu ***@***.***> wrote:
These parsing messages are a nightmare.. I agree, and I would even like to
go a bit further and widdle down the error message in the stack trace a
bit. I think this is a good idea @cthoyt <https://github.com/cthoyt> and
should not be too hard to implement.. I wont do it right now, but if
pressure mounts on this ticket I might be convinced to do it.
—
Reply to this email directly, view it on GitHub
<#1038 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAMMOIIPX5SXXOQXJFDKKLVXUOGBANCNFSM55V6P5AQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
|
Indeed that PR looks like it will get the job done. |
|
@jamesaoverton does this mean if there's a mismatch then it will explode? I think it will still be useful to just get the base ontology information. |
|
My understanding is that if you ask for RDF/XML then only that parser will be used, for the ontology and all of its imports. So if an import is in Turtle (or whatever) that import will fail to parse. |
|
So if the import failing to parse doesn't cause the entire job to break, then this is fine for me. |
|
we definitely want this to fail fast - if O1 imports O2 and O2 is not parseable then loading O1 should fail I think if you want to avoid import closures this needs to be configured separately but this seems like a separate ticket |
Right now, when I get an error from
robot convert, I have to add-vvvto get logs to figure out what is broken in the ontology, but this includes all of the logs from all of the different parsers robot tried. It would be nice to have a--input-formatflag that lets me specify exactly what format the input it is, so this iteration through every possible parser type can be avoided and I can more easily get to my relevant logsE.g., I need to ctrl-f for
org.semanticweb.owlapi.oboformat.OBOFormatOWLAPIParserevery time I am trying to parse an OBO file, and the names are pretty hard to remember for each of these classesThe text was updated successfully, but these errors were encountered: