esp-cpp / espp

C++ components for ESP
https://esp-cpp.github.io/espp/
MIT License
28 stars 9 forks source link

feat(monitor): update monitor to support longer task name printing #233

Closed finger563 closed 1 month ago

finger563 commented 1 month ago

Description

Motivation and Context

Right now the monitor is statically coded to always show (in the table output) 16 character width for the name, and a column for the core id. However, the task name could be shorter or longer (16 is the default configured freertos max task name length), and the core id support may not be compiled in (by default it is not).

This PR updates the table printing to not show core id column if it's not supported, and to update the name column width based on the configured max task name length.

How has this been tested?

Building and running the monitor/example on a QtPy ESP32s3.

Screenshots (if appropriate, e.g. schematic, board, console logs, lab pictures):

Standard config (no core id, max task name length = 16): CleanShot 2024-05-14 at 16 43 24 Longer max name length and core id enabled: CleanShot 2024-05-14 at 16 44 35 Short name length: CleanShot 2024-05-14 at 16 45 48

Types of changes

Checklist:

Software

github-actions[bot] commented 1 month ago

:white_check_mark:Static analysis result - no issues found! :white_check_mark: