Skip to content

Use logger in src/acquisition/* code #1294

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Open
nmdefries opened this issue Sep 28, 2023 · 0 comments · May be fixed by #1299
Open

Use logger in src/acquisition/* code #1294

nmdefries opened this issue Sep 28, 2023 · 0 comments · May be fixed by #1299
Assignees
@dmytrotsko
Labels
acquisition changes acquisition logic code health readability, maintainability, best practices, etc help wanted logs and monitoring logging, monitoring, alerting

Comments

@nmdefries
Copy link
Contributor

Use logger module instead of print and warn statements.
@melange396 melange396 added help wanted code health readability, maintainability, best practices, etc logs and monitoring logging, monitoring, alerting labels Oct 3, 2023
@melange396 melange396 changed the title Use logger in flusurv acquisition script Use logger in src/acquisition/* code Oct 3, 2023
@melange396 melange396 added the acquisition changes acquisition logic label Oct 3, 2023
@dmytrotsko dmytrotsko self-assigned this Oct 3, 2023
@dmytrotsko dmytrotsko linked a pull request Oct 6, 2023 that will close this issue
Replaced print statements with logger #1299
Open
3 tasks
@nmdefries nmdefries mentioned this issue Oct 27, 2023
Open
4 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@dmytrotsko dmytrotsko

Labels
acquisition changes acquisition logic code health readability, maintainability, best practices, etc help wanted logs and monitoring logging, monitoring, alerting
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replaced print statements with logger cmu-delphi/delphi-epidata
3 participants
@melange396 @dmytrotsko @nmdefries