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

[Feature Request]: Pause monitor output (VSC-1294) #1134

Open
project-x51 opened this issue Feb 13, 2024 · 7 comments
Open

[Feature Request]: Pause monitor output (VSC-1294) #1134

project-x51 opened this issue Feb 13, 2024 · 7 comments
Labels
Feature / Enhancement Request Request for Feature/ Enhancement ongoing Ongoing Issue or PR, this label will be used for issue or PR which is to be excluded by stale bot

Comments

@project-x51
Copy link

When I write code that causes the ESP32 to crash due to a panic or similar the monitor output contains a crash dump which I like to read. However typically the code then runs again, crashes and continues to do this in a never ending loop. It very quickly overcomes the buffer for the monitor and then it just scrolls quickly making it difficult to tell what happened. My device has an on/off switch so I just turn it off and then I can read what happened. Not ideal, especially when not all devices have such a switch.

I'd like to be able to pause or stop the monitor using a button on the side of the monitor window. When paused no new data would show until I click the button again to resume monitoring.

As an additional feature it would be cool to also have an auto-pause feature that pauses the log after the crash dump when-ever the device restarts.

Cheers

@project-x51 project-x51 added the Feature / Enhancement Request Request for Feature/ Enhancement label Feb 13, 2024
@github-actions github-actions bot changed the title [Feature Request]: Pause monitor output [Feature Request]: Pause monitor output (VSC-1294) Feb 13, 2024
@SolidStateLEDLighting
Copy link

SolidStateLEDLighting commented Feb 13, 2024

Open up menuconfig and look for "Panic handler behaviour". You may change that behavior by adding delay time before the reboot, or stop the reboot after a panic altogether.

I think that will do what you want.

@project-x51
Copy link
Author

project-x51 commented Feb 13, 2024

Open up menuconfig and look for "Panic handler behaviour". You may change that behavior by adding delay time before the reboot, or stop the reboot after a panic altogether.

I think that will do what you want.

Nice, I like this as a work around/solution to my problem. I still think there is value in being able to pause the monitor log but will use this as my solution for now.

Playing with it I find it strange that the log just stops for the "Panic reboot delay" before outputting the panic information. It would be preferable to output this immediately then delay before restarting.

Also if I use the option "Print registers and halt" then the code stops but nothing is displayed regarding the panic state. Such as the call stack.

Thanks!

@SolidStateLEDLighting
Copy link

You're welcome.

Copy link

This issue has been marked as stale since there are no activities, and this will be closed in 5 days if there are no further activities

@github-actions github-actions bot added the stale Stale PR or Issue label Feb 29, 2024
@github-actions github-actions bot closed this as completed Mar 6, 2024
@brianignacio5 brianignacio5 reopened this Mar 6, 2024
@brianignacio5 brianignacio5 added ongoing Ongoing Issue or PR, this label will be used for issue or PR which is to be excluded by stale bot and removed stale Stale PR or Issue labels Mar 6, 2024
@brianignacio5
Copy link
Collaborator

I'm wondering, do we need a button ?

Here are some keyboard shortcuts for IDF Monitor which allow you to pause for example, (basically stop printing output to terminal): https://docs.espressif.com/projects/esp-idf/en/stable/esp32/api-guides/tools/idf-monitor.html#keyboard-shortcuts

We could add a button in the Espressif Tab, or a command that would send a signal to the IDF Monitor if running, if that helps.

@project-x51
Copy link
Author

project-x51 commented Mar 13, 2024 via email

@VasylMA
Copy link

VasylMA commented Jan 21, 2025

I think it would be good to open the monitor in a separate tab with a short toolbar (clear, copy, pause/resume)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature / Enhancement Request Request for Feature/ Enhancement ongoing Ongoing Issue or PR, this label will be used for issue or PR which is to be excluded by stale bot
Projects
None yet
Development

No branches or pull requests

4 participants