Skip to content
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

Improve logging configuration #47

Open
2 tasks
tloubrieu-jpl opened this issue Jul 26, 2023 · 2 comments
Open
2 tasks

Improve logging configuration #47

tloubrieu-jpl opened this issue Jul 26, 2023 · 2 comments
Labels
i&t.skip Skip I&T of this task/ticket icebox p.could-have task

Comments

@tloubrieu-jpl
Copy link
Member

💡 Description

@alexdunnjpl
Copy link
Contributor

@tloubrieu-jpl why is tqdm actually desirable here? The intent is to show (in ECS logs) periodic progress of various long-runnning queries, which are being consumed concurrently. My gut feeling is that tqdm either wouldn't show up in the way we expect, or would spam the logs in a way that is less legible than the current approach, because tqdm relies on having an interactive terminal with backspacing capability, afaik.

@tloubrieu-jpl
Copy link
Member Author

Hi @alexdunnjpl , none of the items in this ticket is really important.

The first one aims at replacing some ad hoc code with the off-the-self module tqdm of something else. The intention is to simplify the code. Maybe tqdm is not what we need... as you said I used it with interactive terminal or in julyter notebooks, I don't know if we can get something useful out of it for text logs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i&t.skip Skip I&T of this task/ticket icebox p.could-have task
Projects
Status: ToDo
Development

No branches or pull requests

2 participants