-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
output is not always valid ttl #157
Comments
Default to have logging output go to stderr. |
well, for (actual) errors from the log. warnings would be ok. but are there "info" log messages? |
Looks to me like INFO goes to the same logger. |
FMI: Which parts of my code base actually produce logging messages? |
is that speculation or have you seen INFO messages? |
Searching for an import of |
speculation. |
regardless, the logging output should distinguish b/w errors (program must stop) and non-errors (warnings and info). |
It would help if you could provide some details or even a PR. What is the problem specifically? |
thanks. in this case it shouldn't even be a warning. it should fail and the program should exit with a non-zero code. i get around this with my own program wrapper where i fail the program if there is any exception message in the output. not ideal. |
I would disagree that the program should exit - as the error indicates - this is a "WARN". A warning should not cause a failure of the program in my opinion. Isn't it a bit nuclear to stop and fail the whole program because an external ontology could not be imported? |
I agree that warnings should not exit. But Program output should be deterministic. Same output for same input. Here inferencing and validation depends on imported ontologies. |
|
Thanks, happy to take PRs from anyone... |
here's another case where a warning is not acceptable due to malformed (?) uuid.
|
The default logging goes to stdout. If warnings are produced in the process, then the output is not valid ttl.
Possible solutions:
The text was updated successfully, but these errors were encountered: