bugGNU Octave - Bugs: bug #44156, Don't update GUI Variable Browser...

 
 

bug #44156: Don't update GUI Variable Browser Window unless at top-level for performance

Submitted by:  Rik <rik5>
Submitted on:  Mon 02 Feb 2015 05:50:39 AM UTC  
 
Category: GUISeverity: 3 - Normal
Priority: 5 - NormalItem Group: Performance
Status: FixedAssigned to: John W. Eaton <jwe>
Originator Name: Open/Closed: Closed
Release: devOperating System: GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Thu 05 Mar 2015 02:38:07 PM UTC, comment #5:

I checked in the following changeset:

http://hg.savannah.gnu.org/hgweb/octave/rev/793230bb81d4

John W. Eaton <jwe>
Project AdministratorIn charge of this item.
Wed 04 Mar 2015 10:35:00 PM UTC, comment #4:

That seems fine. People who want to see the values of variables in their functions can set breakpoints and query them that way.

Rik <rik5>
Project Administrator
Wed 04 Mar 2015 09:36:27 PM UTC, comment #3:

Ha, well, I always thought I was pretty average human.

In any case, should this be made optional or just unconditional for now? I'm happy to take the easy way out and make it unconditional and see whether anyone complains. If they do, then I'll leave it as an exercise to the complainer to make it an option...

John W. Eaton <jwe>
Project AdministratorIn charge of this item.
Wed 04 Mar 2015 09:23:41 PM UTC, comment #2:

@John: I love it, you're starting to sound human like the rest of us. I have about 40 patches sitting in my Mercurial patch queues, and then another few locations where I just have diffs. I have trouble locating things all the time.

It was in an e-mail to Octave-Maintainers quoted below.

Rik <rik5>
Project Administrator
Wed 04 Mar 2015 09:06:05 PM UTC, comment #1:

Maybe I should know this, but where was that patch posted? I don't know what I did with it...

John W. Eaton <jwe>
Project AdministratorIn charge of this item.
Mon 02 Feb 2015 05:50:39 AM UTC, original submission:

There is no need to update the GUI's Variable Browser if Octave is executing a long calculation. Tests have shown a 60-80% performance increase if the Variable Browser is only updated when at the top_level.

jwe already has a patch for this which seems to work.

Rik <rik5>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Digest:
   bug dependencies.

 

Carbon-Copy List
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by rik5 (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 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 05 Mar 2015 02:38:07 PM UTCjweStatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
    Sat 07 Feb 2015 04:59:35 AM UTCrik5Dependencies-=>bugs #39646 is dependent
    Mon 02 Feb 2015 05:50:56 AM UTCrik5SummaryDon\'t update GUI Variable Browser Window unless at top-lever for performance=>Don't update GUI Variable Browser Window unless at top-level for performance

    Back to the top


    Powered by Savane 3.1-cleanup1