pudb icon indicating copy to clipboard operation
pudb copied to clipboard

Post-mortem debugging in `excepthook` no longer works.

Open inducer opened this issue 2 years ago • 5 comments

See discussion in #549. PRs welcome.

cc @presto-dev

inducer avatar Sep 03 '22 22:09 inducer

Is this still happening? Trying to figure out if post-mortem debugging is expected to work or not.

indigoviolet avatar Nov 02 '23 19:11 indigoviolet

It is if you submit a PR? :slightly_smiling_face:

inducer avatar Nov 02 '23 19:11 inducer

@inducer Makes sense, thank you for confirming. I'm happy to try, but I don't know enough about why https://github.com/inducer/pudb/commit/f77332c5586dccb96cd85b9b92477ea6f749dd65 was made in the first place. Do you have any pointers on what the correct behavior should be, or what should be tested to make sure I don't break something else? Or should I just revert that commit?

indigoviolet avatar Nov 05 '23 01:11 indigoviolet

TBH, I don't remember either. If you can make it work based on your testing, I think that's probably a step in the right direction. It's not a feature I use, so I'm not surprised it bitrotted.

inducer avatar Nov 05 '23 21:11 inducer

I have been missing the post-mortem debugging for a while, really hope to see it back!

jfmoulin avatar Nov 16 '23 18:11 jfmoulin