atom-python-debugger icon indicating copy to clipboard operation
atom-python-debugger copied to clipboard

Enhancement

Open kangshugang opened this issue 6 years ago • 4 comments

Made the following enhancements:

  1. Add variables watch feature
  2. Add call stack view and navigation feature
  3. Add a break point toggle button, which works before and after the debugging started. i.e. Enable turn on/off break points at running time.
  4. Turn on / off the input boxes for arguments and debugging commands.

kangshugang avatar Oct 14 '17 06:10 kangshugang

I think I finally made it through your code. I created a few separate pull requests to merge it in so as to keep changes self-contained. I'm very grateful for your work and contribution! Many thanks! Next time, I'll probably be able to merge faster if you send a few small pull requests instead of a large single pull request.

I haven't yet merged the change related to the callstack. I think it's very worthy and I'd like to add it, but currently the output related to the callstack clobbers other output from the debugger.

dpo avatar Nov 07 '17 03:11 dpo

Dominique,

Thank you very much for your time and effort in reviewing me changes !!!

Actually, I was trying to use Atom as a python programming tool. Indeed, I like Atom, and I like your python debugger. However, without the variable watching function, it is not as convenient as other IDE tools. Basically, I tried to enhance your python debugger to serve my own purposes. After the modification, it works well for me now. Hopefully, these changes could also benefit other people.

Sorry for committing several changes together. Actually, this is the first time that I use GitHub. Thank you for your suggestions, in the future, if I will make any changes, I will try to make small self-contained changes. Thus, reviewing and merging the changes will be easier.

Thanks again and best regards,

Shugang

On Mon, Nov 6, 2017 at 10:55 PM, Dominique [email protected] wrote:

I think I finally made it through your code. I created a few separate pull requests to merge it in so as to keep changes self-contained. I'm very grateful for your work and contribution! Many thanks! Next time, I'll probably be able to merge faster if you send a few small pull requests instead of a large single pull request.

I haven't yet merged the change related to the callstack. I think it's very worthy and I'd like to add it, but currently the output related to the callstack clobbers other output from the debugger.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dpo/atom-python-debugger/pull/33#issuecomment-342368482, or mute the thread https://github.com/notifications/unsubscribe-auth/ASe-oNKcpZnTLidCZBwecBE2HHJuxjYoks5sz9TGgaJpZM4P5SOV .

-- Shugang Kang

Email: [email protected] Phone: 614-214-2186

kangshugang avatar Nov 07 '17 04:11 kangshugang

I now realize that I haven't yet gone over your new panels implementation. That looks very exciting. I'll definitely find some time to look at it carefully later this week!

dpo avatar Nov 07 '17 05:11 dpo

@dpo any progress on this. This PR looks very promising.

luzpaz avatar Nov 20 '18 20:11 luzpaz