patchGNU Octave - Patches: patch #8316, Variable Editor

 
 

patch #8316: Variable Editor

Submitter:  Rüdiger Sonderfeld <ruediger>
Submitted:  Thu 13 Feb 2014 08:53:55 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Duplicate Privacy:  Public
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 11 Jul 2016 07:42:40 PM UTC, comment #4: 

Development of this patch has been continued in patch #8656.

Rüdiger, thanks very much for this contribution.

Michael Barnes has taken it a bit further, It still works fine (except for Java and classdef objects - they'll crash Octave if one tries to open those in the Variable Editor).
I always include it in my Windows builds (until at one day it no longer applies anymore) and I use it fairly frequently.

Closing this report as a duplicate of patch #8656 has seen more recent development. Hopefully someone will pick up the current state from there.

Philip Nienhuis <philipnienhuis>
Group Member
Sat 15 Feb 2014 12:43:09 AM UTC, comment #3: 

I've updated the patch.  The variable editor is now refreshed every time `octave_link::set_workspace' gets called (i.e., every time it the octave interpreter waits for input).  I'm open to better suggestions.

I fixed the thread safety issue and many other issues.

Remaining

  • Paste not working
  • Inserting/removing rows/cols (and Delete) not working (some untested code for it but no GUI interface)
  • No support for structure arrays
  • Plotting doesn't work for cell arrays.
  • "Variable from selection" always uses "unnamed" as variable name.
  • No window splitting
  • No data brush
  • No printing
  • No undo support
  • No sort
  • Some icons would be nice.


All in all it should be usable.

(file #30552)

Rüdiger Sonderfeld <ruediger>
Sat 15 Feb 2014 12:32:54 AM UTC, comment #2: 
Rüdiger Sonderfeld <ruediger>
Sat 15 Feb 2014 12:32:23 AM UTC, comment #1: 
Rüdiger Sonderfeld <ruediger>
Thu 13 Feb 2014 08:53:55 PM UTC, original submission:  

This patch adds a basic Variable Editor to the GUI.  It is not finished but I wanted some input before continuing with it.

The biggest issue right now is that the editor does not detect changes happening to the variable inside Octave.  It will continue displaying the old value.  I don't know if there is a way in Octave to intercept changes to certain variables or how this could be implemented.  The simplest way would be to update the display completely once `octave_link::set_workspace' gets called.  But this would only update the view when Octave has finished and is waiting for new input.  Maybe that would be good enough at least for an initial release.

There are some other issues

  • Paste not working
  • Inserting/removing rows/cols not working
  • No support for structure arrays (completely untested)
  • No subeditor opens for internal structures (but the input is eval'd so it is somewhat usable at the moment)
  • Plotting doesn't work for cell arrays.
  • Delete does not support shifting and always inserts 0 (should be [] for cell arrays)
  • No variable from selection or transpose feature
  • No window splitting
  • No data brush
  • No printing
  • No undo support
  • No sort
  • ...


And a potential thread safety issue.  I call `symbol_table::varval' in `main_window' from the GUI thread to get the `octave_value'.  It could be safely passed via signal but that would require registering `octave_value' as Qt metatype and I don't know what implications that would have (See http://qt-project.org/doc/qt-4.8/qmetatype.html#qRegisterMetaType).

Rüdiger Sonderfeld <ruediger>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #30552:  variable-editor-2.patch added by ruediger (57KiB - text/x-patch)
file #30545:  variable-editor.patch added by ruediger (45KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by ruediger (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-07-11 philipnienhuis StatusNone Duplicate
        Open/ClosedOpen Closed
    2014-02-15 ruediger Attached File- Added variable-editor-2.patch, #30552
    2014-02-13 ruediger Attached File- Added variable-editor.patch, #30545

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code