You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When the query command runs an update, it has to parse the resulting ontology again. It looks in the CommandState for a catalog that may have been used at the beginning, so that it can properly parse the ontology. However, merge does not put the catalog path into the state. Commands that expect a single ontology use updateInputOntology to get the current state and ontology; in this case if there is a catalog file the path is stored in the state.
But merge uses getInputOntologies, which returns a list of ontologies rather than an updated state.
I've been trying to decide on the best way to store the catalog path in this case, but wanted to go ahead and make the issue so it doesn't get lost. I'm only just now realizing that this command has been reloading imports off the web instead of locally partway through.
This problem may exist for other commands as well; I haven't checked.
The text was updated successfully, but these errors were encountered:
This took me a while to figure out. In a chain of commands like this:
When the
query
command runs anupdate
, it has to parse the resulting ontology again. It looks in theCommandState
for a catalog that may have been used at the beginning, so that it can properly parse the ontology. However,merge
does not put the catalog path into the state. Commands that expect a single ontology use updateInputOntology to get the current state and ontology; in this case if there is a catalog file the path is stored in the state.But
merge
uses getInputOntologies, which returns a list of ontologies rather than an updated state.I've been trying to decide on the best way to store the catalog path in this case, but wanted to go ahead and make the issue so it doesn't get lost. I'm only just now realizing that this command has been reloading imports off the web instead of locally partway through.
This problem may exist for other commands as well; I haven't checked.
The text was updated successfully, but these errors were encountered: