-
Notifications
You must be signed in to change notification settings - Fork 15
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
Logging with origami. #38
Comments
@fristonio Are you thinking of adding something on the lines of Cloud-CV/origami-daemon#4? |
Yeah, mostly like this. This way we can log better logs without needing the user to configure anything. |
Yeah sounds good. Also origami-lib should send logs with a certain label to the Origami web application for display to the user of the demo. |
Yeah, I have it in the todo list for origami-lib, but the thing is I didn't wanna mangle too much with the code that's already there in the frontend. We can do this easily once we have a basic working app with us. |
Sounds good! 👍 |
I was thinking that we can provide origami-lib with a logger functionality which will set up the logger for the demo. So a single method will bootstrap the python
logging
library which can then be used by demo user.This will help us when we are deploying demos to our server too, suppose user want to check the logs for his demo the logger will store them in a file which can then be provided to demo owner.
Thoughts on this?
@deshraj @uttu357 @AvaisP @dexter1691 @vpn1997
The text was updated successfully, but these errors were encountered: