Skip to content

Add LogMessageViewer#46

Merged
defunctzombie merged 1 commit intofoxglove:mainfrom
flypyka:add-pyka-text-log-extension
May 28, 2025
Merged

Add LogMessageViewer#46
defunctzombie merged 1 commit intofoxglove:mainfrom
flypyka:add-pyka-text-log-extension

Conversation

@kmmoore
Copy link
Contributor

@kmmoore kmmoore commented May 27, 2025

Adds an extension that displays any foxglove.Log messages received during the entire displayed timeline. Allows filtering based on severity and contents. Clicking a message jumps the playhead to that timestamp

Adds an extension that displays any foxglove.Log messages received
during the entire displayed timeline. Allows filtering based on
severity and contents. Clicking a message jumps the playhead to that
timestamp
@defunctzombie
Copy link
Contributor

@kmmoore Since we already have a log viewer built-in, what's the main thing you found lacking with it? The ability to view logs across the entire timeline and click-to-seek?

While testing this did you run into memory issues or performance? One concern we have with tackling this with built-in is the implication it has to memory use if someone has lots of logs.

@kmmoore
Copy link
Contributor Author

kmmoore commented May 27, 2025

Hi thanks for taking a look! The main features I wanted were:

  • The ability to see all log messages that occurred in the displayed time range (the built in extension seems to only display them once you play past the point where the message is received).
  • The ability to click on a message and jump the playhead to that location in the timeline.
  • The ability to fuzzy-filter log messages (a la fzf, or sublime text).

I didn't run in to any performance/memory issues in the logs I tested (although I didn't test any truly massive logs). The extension does have a configurable limit on the maximum number of messages it will load to hopefully prevent pathological cases.

@defunctzombie defunctzombie merged commit f5d185c into foxglove:main May 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants