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

Added auto save/load of watch expressions #150

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lechat
Copy link
Contributor

@lechat lechat commented Jun 24, 2015

No description provided.

@inducer
Copy link
Owner

inducer commented Jun 28, 2015

Thanks for your contribution! I'm not completely sure about this functionality though. I think of a watch expression as something you add while poking around in a single function--and so having them saved globally sort of assumes that you're going to be poking around the same function after the next restart of the debugger.

I'm also not quite sure I like how the code is written, since the saving happens as part of what is supposed to be a UI update.

The pain point solved by this is that retyping watch expressions is cumbersome and takes time. An alternate UI design might be to keep a history of watch expressions that a user typed in in the past and to offer those up when a new watch expression is being created. It would probably be nice to make it efficient to quickly add multiple expressions from that history in one go. What do you think?

@maho
Copy link

maho commented Oct 16, 2017

I think that option to have persistent watch expressions is very important for me. I'm entering the same function over and over and over, and writing the same things is annoying.

Also, it would be nice if console remember history (for same reasons) and panels remember their sizes (but it's BTW)

@asmeurer
Copy link
Collaborator

I think you could say the same thing about breakpoints. Once you're done debugging something you don't really need your breakpoints any more. But while debugging, it's good to have them saved, because you typically have to restart the debugger many times. So I think saving them is nice, as long as it's also easy to clear them once you no longer need them.

@maho
Copy link

maho commented Oct 30, 2017

Breakpoints are auto-saved out of the box.

@igordejanovic
Copy link

@lechat Thanks a lot for this contribution. It's a saver. I had to start debugger over and over with several watch expressions. Having to type them each time was really annoying.

@zhou13
Copy link

zhou13 commented Jan 28, 2020

I also like to see this in the upstream, which makes the debugger much useful for me. When I debug a function, I need to restart the program multiple times to see if my modification is effective. In addition, I need to track multiple variables inside a function. Retyping those variables in a watch every time is not acceptable for me. For now, I need to print them on the screen. I wonder what is the designed workflow of pudb currently for this kind of debug.

Base automatically changed from master to main March 8, 2021 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants