bugGNU Octave - Bugs: bug #44455, GUI: workspace pane remains empty...

 
 

bug #44455: GUI: workspace pane remains empty during debugging

Submitted by:  Philip Nienhuis <philipnienhuis>
Submitted on:  Fri 06 Mar 2015 06:30:10 PM UTC  
 
Category: GUISeverity: 3 - Normal
Priority: 5 - NormalItem Group: Regression
Status: FixedAssigned to: John Donoghue <lostbard>
Originator Name: Philip NienhuisOpen/Closed: Closed
Release: 4.0.0Operating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Tue 02 Jun 2015 04:37:34 PM UTC, comment #10:

Retagging release from 4.0.0-rc1 to 4.0.0.

John W. Eaton <jwe>
Project Administrator
Mon 23 Mar 2015 11:56:28 AM UTC, comment #9:

Closing report

John Donoghue <lostbard>
Project MemberIn charge of this item.
Fri 20 Mar 2015 12:45:30 PM UTC, comment #8:

Pushed http://hg.savannah.gnu.org/hgweb/octave/rev/f7846f0ea6db

John Donoghue <lostbard>
Project MemberIn charge of this item.
Thu 19 Mar 2015 02:33:23 PM UTC, comment #7:

Patch to update workspace also when in debug mode.

(file #33398)

John Donoghue <lostbard>
Project MemberIn charge of this item.
Sat 14 Mar 2015 08:09:41 PM UTC, comment #6:

So we fully agree there (see my comment #2).

Maybe workspace/directory updates should be triggered whenever the CLI is waiting for user input? (some command or "dbnext" or whatever debug stmt)

Philip Nienhuis <philipnienhuis>
Project Member
Sat 14 Mar 2015 06:13:55 PM UTC, comment #5:

To my personal opinion, being able to see function variables in the workspace widget at a breakpoint is more useful than avoiding unnecessary updates of this widgets.

Torsten <ttl>
Project Member
Sun 08 Mar 2015 06:54:05 PM UTC, comment #4:

I know Torsten, but

  • it is a bit beside the point (which is somewhat needless updating of panes while Octave is doing something)
  • I can follow that it might help but that to do that every time before and after I run a script.....
Philip Nienhuis <philipnienhuis>
Project Member
Sun 08 Mar 2015 12:49:20 PM UTC, comment #3:

Concerning the directory pane: You can switch off the synchronization of the octave directory and the directory shown in the browser (settings dialog, pane "File Browser").

Torsten <ttl>
Project Member
Sun 08 Mar 2015 11:04:14 AM UTC, comment #2:

"regression" ok, that's somewhat arguable.
IMO the regression is that the workspace viewer became unusable when it is the most useful/needed, i.e., while debugging.

BTW it's not only updating the workspace pane that takes time while scripts are running; the directory pane is also a hog in this respect, maybe even worse, continuously updating when Octave changes directory in scripts.

Philip Nienhuis <philipnienhuis>
Project Member
Sun 08 Mar 2015 06:02:39 AM UTC, comment #1:

Confirmed here on Linux. You're probably right about the change making the workspace view not update. Not sure that I'd call this a regression since that was the point of the change, to avoid updating the workspace view except when control returns to the command prompt. I guess the question is can we keep that change but also update the workspace view when a breakpoint prompt is hit?

Mike Miller <mtmiller>
Project Administrator
Fri 06 Mar 2015 06:30:10 PM UTC, original submission:

After starting a debugging session as one of the first things after starting Octave (when no variables have been defined yet) I can see no variables / objects / etc. in the workspace pane. I'd expect to see/in earlier Octave versions I saw, the variables belonging to the debugged function/subfunction.

"whos" gives me a complete listing.

Investigating further, when defining just one var in the terminal ("a = 10"), that workspace remains the only visible one during a debug session.

Is this related to cs 793230bb81d4 "don't update workspace viewer unless at top level (bug #44156)"?

Pic of Octave's window attached

Philip Nienhuis <philipnienhuis>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #33398:  debugfix.patch added by lostbard (6KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by lostbard (Updated the item)
  • -unavailable- added by ttl (Posted a comment)
  • -unavailable- added by mtmiller (Posted a comment)
  • -unavailable- added by philipnienhuis (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 10 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 02 Jun 2015 04:37:34 PM UTCjweRelease4.0.0-rc1=>4.0.0
    Mon 23 Mar 2015 11:56:28 AM UTClostbardAssigned toNone=>lostbard
      Open/ClosedOpen=>Closed
    Mon 23 Mar 2015 11:56:27 AM UTClostbardStatusReady For Test=>Fixed
    Fri 20 Mar 2015 12:45:30 PM UTClostbardStatusPatch Submitted=>Ready For Test
    Thu 19 Mar 2015 02:33:23 PM UTClostbardAttached File-=>Added debugfix.patch, #33398
      StatusConfirmed=>Patch Submitted
    Sun 08 Mar 2015 06:02:39 AM UTCmtmillerStatusNone=>Confirmed
      Operating SystemMicrosoft Windows=>Any
    Fri 06 Mar 2015 06:30:10 PM UTCphilipnienhuisAttached File-=>Added debug_wo_workspace.png, #33266

    Back to the top


    Powered by Savane 3.1-cleanup1