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

[FEAT] create a DEBUG on/off capability #23

Open
Olzeke51 opened this issue Jun 9, 2024 · 1 comment
Open

[FEAT] create a DEBUG on/off capability #23

Olzeke51 opened this issue Jun 9, 2024 · 1 comment
Labels
triage New issue needs review

Comments

@Olzeke51
Copy link

Olzeke51 commented Jun 9, 2024

Objective

Since DEBUG creates a lot of feedback, consuming 'log files' space [?10MB limit];
create a system-wide variable that can turn on & off a DEBUG flag for logging utils to use

Initial Implementation Requirements

these are what I could find , but couldn't determine how/where ~/.config/neon/neon.yaml fit into the picture:

(venv) neon@neon:~/venv/lib/python3.10/site-packages/neon_utils$ ls log*.py
log_utils.py  logger.py

Other Considerations

? having to check the flag for each entry into the file could slow it down, overflow a buffer/cache ??
every 10 secs in the skill log, bus log was 9+MB !!! {logs are archived in my BUG issue}

2024-06-08 05:13:40.371 - skills - ovos_bus_client.util.scheduler:check_state:186 - DEBUG - Call scheduled event: skill-ovos-homescreen.openvoiceos:skill-ovos-homescreen.openvoiceosupdate_dt
2024-06-08 05:13:50.390 - skills - ovos_bus_client.util.scheduler:check_state:186 - DEBUG - Call scheduled event: skill-ovos-homescreen.openvoiceos:skill-ovos-homescreen.openvoiceosupdate_dt
2024-06-08 05:14:00.411 - skills - ovos_bus_c
* * notice it cut-off in mid textr !
@Olzeke51 Olzeke51 added the triage New issue needs review label Jun 9, 2024
@NeonDaniel
Copy link
Member

This is read from configuration currently (user values set in ~/.config/neon/neon.yaml by default). The configuration options are noted in ovos-utils which is where the LOG class is implemented

@NeonDaniel NeonDaniel transferred this issue from NeonGeckoCom/NeonCore Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage New issue needs review
Projects
Status: Low Priority
Development

No branches or pull requests

2 participants