Skip to content

feat: Extend python debugging monitor functionality (keyword filter / suppress)#810

Merged
daveallie merged 2 commits intocrosspoint-reader:masterfrom
jpirnay:feat-extend_monitor
Feb 10, 2026
Merged

feat: Extend python debugging monitor functionality (keyword filter / suppress)#810
daveallie merged 2 commits intocrosspoint-reader:masterfrom
jpirnay:feat-extend_monitor

Conversation

@jpirnay
Copy link
Contributor

@jpirnay jpirnay commented Feb 9, 2026

Summary

  • I needed the ability to filter and or suppress debug messages containig certain keywords (eg [GFX] for render related stuff)
  • Update of debugging_monitor.py script for development work

Additional Context

usage: debugging_monitor.py [-h] [--baud BAUD] [--filter FILTER] [--suppress SUPPRESS] [port]

ESP32 Monitor with Graph

positional arguments:
  port                 Serial port

options:
  -h, --help           show this help message and exit
  --baud BAUD          Baud rate
  --filter FILTER      Only display lines containing this keyword (case-insensitive)
  --suppress SUPPRESS  Suppress lines containing this keyword (case-insensitive)
  • plus a couple of platform specific defaults (port, pip style)

AI Usage

While CrossPoint doesn't have restrictions on AI tools in contributing, please be transparent about their usage as it
helps set the right context for reviewers.

Did you use AI tools to help write this code? NO

jonasdiemer
jonasdiemer requested changes Feb 10, 2026
View reviewed changes
Copy link
Contributor

@jonasdiemer jonasdiemer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Superficially tested, works on my machine.

To suppress the drawing updates, we should probably prepend [GFX] on all lines - currently, only the first one of these is filtered:

[08:57:50] [GFX] Time = 450 ms from clearScreen to displayBuffer
[08:57:50]   Writing frame buffer to BW RAM (48000 bytes)...
[08:57:50]   BW RAM write complete (12 ms)
[08:57:50]   Powering on display 0x1F (fast refresh)...
[08:57:50]   Waiting for display refresh...
[08:57:51]   Wait complete: fast (562 ms)
[08:57:51]   Writing frame buffer to RED RAM (48000 bytes)...
[08:57:51]   RED RAM write complete (12 ms)
[08:57:51] [GFX] Time = 25 ms from clearScreen to displayBuffer
[08:57:51]   Writing frame buffer to BW RAM (48000 bytes)...
[08:57:51]   BW RAM write complete (12 ms)
[08:57:51]   Powering on display 0xDF (fast refresh)...
[08:57:51]   Waiting for display refresh...
[08:57:52]   Wait complete: fast (644 ms)
[08:57:52]   Writing frame buffer to RED RAM (48000 bytes)...
[08:57:52]   RED RAM write complete (12 ms)

also noticed the "wait complete" message has a different color...

@daveallie daveallie merged commit 0c2df24 into crosspoint-reader:master Feb 10, 2026
5 checks passed
lukestein pushed a commit to lukestein/crosspoint-reader that referenced this pull request Feb 10, 2026
@jpirnay @lukestein
feat: Extend python debugging monitor functionality (keyword filter /…
7441103 
… suppress) (crosspoint-reader#810)

## Summary

* I needed the ability to filter and or suppress debug messages
containig certain keywords (eg [GFX] for render related stuff)
* Update of debugging_monitor.py script for development work

## Additional Context
```
usage: debugging_monitor.py [-h] [--baud BAUD] [--filter FILTER] [--suppress SUPPRESS] [port]

ESP32 Monitor with Graph

positional arguments:
  port                 Serial port

options:
  -h, --help           show this help message and exit
  --baud BAUD          Baud rate
  --filter FILTER      Only display lines containing this keyword (case-insensitive)
  --suppress SUPPRESS  Suppress lines containing this keyword (case-insensitive)
```
* plus a couple of platform specific defaults (port, pip style)
---

### AI Usage

While CrossPoint doesn't have restrictions on AI tools in contributing,
please be transparent about their usage as it
helps set the right context for reviewers.

Did you use AI tools to help write this code? NO
lukestein pushed a commit to lukestein/crosspoint-reader that referenced this pull request Feb 10, 2026
@jpirnay @lukestein
feat: Extend python debugging monitor functionality (keyword filter /…
d0a0c52 
… suppress) (crosspoint-reader#810)

## Summary

* I needed the ability to filter and or suppress debug messages
containig certain keywords (eg [GFX] for render related stuff)
* Update of debugging_monitor.py script for development work

## Additional Context
```
usage: debugging_monitor.py [-h] [--baud BAUD] [--filter FILTER] [--suppress SUPPRESS] [port]

ESP32 Monitor with Graph

positional arguments:
  port                 Serial port

options:
  -h, --help           show this help message and exit
  --baud BAUD          Baud rate
  --filter FILTER      Only display lines containing this keyword (case-insensitive)
  --suppress SUPPRESS  Suppress lines containing this keyword (case-insensitive)
```
* plus a couple of platform specific defaults (port, pip style)
---

### AI Usage

While CrossPoint doesn't have restrictions on AI tools in contributing,
please be transparent about their usage as it
helps set the right context for reviewers.

Did you use AI tools to help write this code? NO
lukestein pushed a commit to lukestein/crosspoint-reader that referenced this pull request Feb 10, 2026
@jpirnay @lukestein
feat: Extend python debugging monitor functionality (keyword filter /…
24872d7 
… suppress) (crosspoint-reader#810)

## Summary

* I needed the ability to filter and or suppress debug messages
containig certain keywords (eg [GFX] for render related stuff)
* Update of debugging_monitor.py script for development work

## Additional Context
```
usage: debugging_monitor.py [-h] [--baud BAUD] [--filter FILTER] [--suppress SUPPRESS] [port]

ESP32 Monitor with Graph

positional arguments:
  port                 Serial port

options:
  -h, --help           show this help message and exit
  --baud BAUD          Baud rate
  --filter FILTER      Only display lines containing this keyword (case-insensitive)
  --suppress SUPPRESS  Suppress lines containing this keyword (case-insensitive)
```
* plus a couple of platform specific defaults (port, pip style)
---

### AI Usage

While CrossPoint doesn't have restrictions on AI tools in contributing,
please be transparent about their usage as it
helps set the right context for reviewers.

Did you use AI tools to help write this code? NO
jdk2pq added a commit to jdk2pq/crosspoint-reader that referenced this pull request Feb 11, 2026
@jdk2pq
Merge branch 'master' into fix/account-for-nbsp-character-as-non-brea…
dc8e1a8 
…king-space

* master:
  fix: Reduce MIN_SIZE_FOR_POPUP to 10KB (crosspoint-reader#809)
  docs: Update USER_GUIDE.md (crosspoint-reader#817)
  fix: Prevent sleeping when in OPDS browser / downloading books (crosspoint-reader#818)
  feat: Extend python debugging monitor functionality (keyword filter / suppress) (crosspoint-reader#810)
  docs: Update USER_GUIDE.md (crosspoint-reader#808)
  feat: Connect to last wifi by default (crosspoint-reader#752)
jdk2pq added a commit to jdk2pq/crosspoint-reader that referenced this pull request Feb 13, 2026
@jdk2pq
Merge branch 'master' into perf/improve-large-css-file-handling
50e024b 
* master: (25 commits)
  fix: Reduce MIN_SIZE_FOR_POPUP to 10KB (crosspoint-reader#809)
  docs: Update USER_GUIDE.md (crosspoint-reader#817)
  fix: Prevent sleeping when in OPDS browser / downloading books (crosspoint-reader#818)
  feat: Extend python debugging monitor functionality (keyword filter / suppress) (crosspoint-reader#810)
  docs: Update USER_GUIDE.md (crosspoint-reader#808)
  feat: Connect to last wifi by default (crosspoint-reader#752)
  feat: use natural sort in file browser (crosspoint-reader#722)
  fix: issue if book href are absolute url and not relative to server (crosspoint-reader#741)
  feat: unify navigation handling with system-wide continuous navigation (crosspoint-reader#600)
  feat: Add Italian hyphenation support (crosspoint-reader#584)
  feat: Add percentage support to CSS properties (crosspoint-reader#738)
  Use GITHUB_REF_NAME over GITHUB_HEAD_REF in release candidate workflow
  Move release candidate workflow to manual dispatch
  fix: Allow OTA update from RC build to full release (crosspoint-reader#778)
  refactor: Rename "Embedded Style" to "Book's Embedded Style" (crosspoint-reader#746)
  perf: optimize drawPixel() (crosspoint-reader#748)
  feat: wakeup target detection (crosspoint-reader#731)
  fix: Scrolling page items calculation (crosspoint-reader#716)
  feat: optimize fillRectDither (crosspoint-reader#737)
  fix: increase lyra sideButtonHintsWidth to 30 (crosspoint-reader#727)
  ...
Unintendedsideeffects pushed a commit to Unintendedsideeffects/crosspoint-reader that referenced this pull request Feb 17, 2026
@jpirnay
feat: Extend python debugging monitor functionality (keyword filter /…
e9d21f7 
… suppress) (crosspoint-reader#810)

## Summary

* I needed the ability to filter and or suppress debug messages
containig certain keywords (eg [GFX] for render related stuff)
* Update of debugging_monitor.py script for development work

## Additional Context
```
usage: debugging_monitor.py [-h] [--baud BAUD] [--filter FILTER] [--suppress SUPPRESS] [port]

ESP32 Monitor with Graph

positional arguments:
  port                 Serial port

options:
  -h, --help           show this help message and exit
  --baud BAUD          Baud rate
  --filter FILTER      Only display lines containing this keyword (case-insensitive)
  --suppress SUPPRESS  Suppress lines containing this keyword (case-insensitive)
```
* plus a couple of platform specific defaults (port, pip style)
---

### AI Usage

While CrossPoint doesn't have restrictions on AI tools in contributing,
please be transparent about their usage as it
helps set the right context for reviewers.

Did you use AI tools to help write this code? NO
@daveallie daveallie mentioned this pull request Feb 19, 2026
Merged
@jpirnay jpirnay deleted the feat-extend_monitor branch February 20, 2026 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@daveallie daveallie daveallie approved these changes

@jonasdiemer jonasdiemer jonasdiemer approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jpirnay @daveallie @jonasdiemer