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.

Sign up for GitHub

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

Jump to bottom

Add logging #139

Open
JanSchankin opened this issue May 11, 2022 · 0 comments
Open

Add logging #139

JanSchankin opened this issue May 11, 2022 · 0 comments
Assignees
@JanSchankin
Labels
enhancement New feature or request

Comments

@JanSchankin
Copy link
Contributor

AOP comes with a downside: If an aspect is not called due to a bug, this bug is not visible, since no code is executed, that could notice the bug and throw an Exception, or print an error message.
As a result, DeepSampler tends to do nothing, if some configuration errors exist. This is quite hard to debug. This could be eased by logging.

Since DeepSampler is an API that must not force the usage of a specific logging API, we must find a possibility to allow a form of logging, that fits into an existing logging environment, and comes without additional dependencies.
@JanSchankin JanSchankin added the enhancement New feature or request label May 11, 2022
@JanSchankin JanSchankin self-assigned this May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JanSchankin JanSchankin

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@JanSchankin