Skip to content

Add calloop::sources=debug to default log filters#23763

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
BigWingBeat:logging_filter
Apr 12, 2026
Merged

Add calloop::sources=debug to default log filters#23763
alice-i-cecile merged 1 commit intobevyengine:mainfrom
BigWingBeat:logging_filter

Conversation

@BigWingBeat
Copy link
Copy Markdown
Contributor

Objective

When setting the default log level to trace (e.g. with RUST_LOG=trace), one of the log messages that gets enabled looks like [calloop] Processing events for source type .... This message is logged at the trace level 1-3 times every frame, making it extremely spammy. Although the purpose of trace-level logs is to be verbose, logging the same unactionable thing every frame is excessive.
While normally there would be a bunch of stuff that always gets logged every frame, the default filters provided by LogPlugin exclude all of that at every log level, except for this one thing which slips through.

Solution

Add the offending module to the default filters.

Testing

Manually tested with the logs example and various RUST_LOG values.

@alice-i-cecile alice-i-cecile added A-Windowing Platform-agnostic interface layer to run your app in C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Apr 12, 2026
@alice-i-cecile alice-i-cecile added A-Diagnostics Logging, crash handling, error reporting and performance analysis D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Apr 12, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Apr 12, 2026
Merged via the queue into bevyengine:main with commit 891d4bb Apr 12, 2026
53 checks passed
@BigWingBeat BigWingBeat deleted the logging_filter branch April 13, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Diagnostics Logging, crash handling, error reporting and performance analysis A-Windowing Platform-agnostic interface layer to run your app in C-Usability A targeted quality-of-life change that makes Bevy easier to use D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants