Changes between Initial Version and Version 3 of Ticket #453


Ignore:
Timestamp:
Jul 12, 2015 8:05:13 PM (9 years ago)
Author:
butler
Comment:

Indeed ticket #424 was closed too hastily. It turns out there were several problems that still needed to be addressed.

  1. The main problem as identified in that ticket were EndEdit and PaintBackground having the wrong number of parameters. GridCellEditor was overwritten in data_processor and init and EndEdit methods were overwritten. However PaintBackground was not. While the error (too many arguments given) continued to be generated every keystroke it no longer prevented the cell from being edited. This was fixed by overriding the PaintBackground class, addding it to the new GridCellEditor class in data_processor (a bit tricker since it needed to call not the parent version but the parent's parent). No errors seem to be generated on editing any more.
  2. A few of the Grid events being used were not bound to the local methods in the process of the cleanup. In particular EVT_GRID_LABEL_RIGHT_CLICK used to bring up the context menu that allows for insertion of a column and EVT_GRID_LABEL_LEFT_CLICK that is needed to properly set the data ranges for plotting. These have been added back calling the appropriate code.
  3. In the process of removing the Edit_Menu item the on_open_menu method's section dealing with that menu item was not commented out. Thus any later menu would get the id such that when opened and that method called would execute the section written for the Edit menu and raise an exception. The removal of the menu item was done so as to be able to be able to easily add it back later if we choose to. Thus rather than commenting the section out, the self.edit = NONE was replaced wth an unused handle self.edit = wx.Menu() preventing the next menu item to get confused. This means that re-implementing the menu item should just be a question of commenting out the self.edit line and uncommenting the line beneath it which calls the add_edit_menu method. Meanwhile everything seems to now work without generating any errors without the menu being added.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #453

    • Property Owner set to butler
    • Property Status changed from new to accepted
  • Ticket #453 – Description

    initial v3  
    11ESS Win 7 Ver 3.1.0 Build 126 
    22 
    3 Whilst noting Ticket #431 (http://trac.sasview.org/ticket/431), it is the case that as things stand you can no longer add the parameter uncertainties to the Grid Window, which kind of negates the usefulness of Batch Mode. 
     3Whilst noting Ticket #431, it is the case that as things stand you can no longer add the parameter uncertainties to the Grid Window, which kind of negates the usefulness of Batch Mode.