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

Disable logging messages for RL agent #37

Merged
merged 1 commit into from
Mar 23, 2024

Conversation

visitorckw
Copy link
Collaborator

During gameplay and training, the RL agent would print Q values for available moves as part of its move selection process. However, this information, mainly useful for debugging purposes, is not required for typical usage scenarios. To enhance readability and streamline output, this patch disables the logging output by default. The related messages will now only appear if the verbose option is explicitly enabled during compilation. This change reduces unnecessary verbosity during gameplay and training, improving overall user experience.

See: #35

During gameplay and training, the RL agent would print Q values for
available moves as part of its move selection process. However, this
information, mainly useful for debugging purposes, is not required for
typical usage scenarios. To enhance readability and streamline output,
this patch disables the logging output by default. The related messages
will now only appear if the verbose option is explicitly enabled during
compilation. This change reduces unnecessary verbosity during gameplay
and training, improving overall user experience.
@visitorckw visitorckw requested a review from jserv March 21, 2024 16:41
@jserv jserv merged commit 9ebe4cd into jserv:main Mar 23, 2024
2 checks passed
@visitorckw visitorckw deleted the disable-logging-output branch March 23, 2024 03:34
@visitorckw visitorckw restored the disable-logging-output branch March 23, 2024 03:34
@visitorckw visitorckw deleted the disable-logging-output branch March 23, 2024 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants