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

dvc disables logging of other packages #3344

Closed
ttekampe opened this issue Feb 17, 2020 · 1 comment · Fixed by #3345
Closed

dvc disables logging of other packages #3344

ttekampe opened this issue Feb 17, 2020 · 1 comment · Fixed by #3345
Assignees
Labels
bug Did we break something? p2-medium Medium priority, should be done, but less important

Comments

@ttekampe
Copy link
Contributor

DVC version is 0.80.0 and I am running Linux. The package was installed from AUR

To reproduce the issue run the following few lines of code

import logging

LOGGER = logging.getLogger(__name__)

print(LOGGER.disabled)

import dvc
print(LOGGER.disabled)

which will output

False
True

The reason for this is the usage of logging.config.dictConfig here without adding "disable_existing_loggers": False to the dict

@triage-new-issues triage-new-issues bot added the triage Needs to be triaged label Feb 17, 2020
@efiop
Copy link
Contributor

efiop commented Feb 17, 2020

Hi @ttekampe !

Good catch! And thanks for the PR!

Please be aware that AUR package is not officially supported, so consider using some other package.

@efiop efiop added bug Did we break something? p2-medium Medium priority, should be done, but less important labels Feb 18, 2020
@triage-new-issues triage-new-issues bot removed the triage Needs to be triaged label Feb 18, 2020
@gurobokum gurobokum self-assigned this Feb 24, 2020
@efiop efiop assigned gurobokum and unassigned gurobokum Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Did we break something? p2-medium Medium priority, should be done, but less important
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants