PlotProperties getting QPromptCellWidget instead of cell widget

[remram44]

From CDAT/VisTrails#30

It seems that PlotProperties is updated from ProjectController#update_workflow() just after execute_plot() and this is not safe. The PlotProperties gets the QPromptCellWidget instead of the new cell widget.

A workaround can be added for 2.2 but this should definitely be fixed later.

[doutriaux1]

@remram44 actually the work around doesn't really work when I apply it (CDAT/VisTrails#30) it does not break anymore, but doesn't seem to produce any plot any longer. @vvpalav or @hgohil2805 can you take a look please?

[hgohil2805]

@remram44 This is caused by the cell not being deleted correctly and because of the changes that we made last week of setting it to none in the delete later method. I had encountered similar exception with the changes that I had shown you before and I had used a catch all which worked fine (not sure if you remember the changes).

Until we find the the object which is keeping the reference to the cell and remove it while deleting, I feel keeping a catch all should be fine as incorrect cell removal should probably be the only thing which causes the exception to be thrown .

[doutriaux1]

@hgohil2805 not sure the catch all is actually working in this case, it does prevent it form failing, but nothing gets drawn in the spreadsheet. Now that might be something else that prevents the plot from appearing I'm not positive.

[remram44]

@hgohil2805 I have no idea what you're talking about and disagree on all points.

[remram44]

@doutriaux1 I don't run into the issue, with either workaround or none at all. What system are you using?
The patch in CDAT/VisTrails#30 should prevent the traceback you gave from happening; it might interfere with the the PlotProperties, but really can't prevent plotting.

[doutriaux1]

It's very hard to reproduce. Has to come from diagnostics and taor diagrmasn. Ask @mcenerney1 or @painter1 for steps and location of data. Because I can't reproduce with uvcmetrics test data. I need @mcenerney1 special data.

[mcenerney1]

I really need help on this one and I'll help out in any way possible. On aims1 in my area (mcenerney1) there are two files cam35_data.tar (model data) and taylor_obs_data.tar (observation data). You'll need to download them to reproduce the issue. Documentation on running the AMWG diagnostics from the gui can be found at http://uvcdat-dev.llnl.gov/documentation/diagnostics/diagnostics.html
To run from the command line, you'll need uvcmetrics. In the directory uvcmetrics/src/python/frontend run
diags.py --model path=$HOME/uvcmetrics_test_data/cam35_data/,climos=yes
--obs path=$HOME/uvcmetrics_test_data/obs_data/,filter='f_startswith("NCEP")',climos=yes
--outputdir $HOME/Documents/Climatology/ClimateData/diagout/ --package AMWG --sets 14
--seasons JAN --plots yes --vars T --varopts '200 mbar'

Once again I'll do whatever it takes to get this resolved; email, phone calls, telepathy. Thanks
Jim

[doutriaux1]

@mcenerney1 they need instructions for gui I fixed command line already. @painter1 merged it.
But gui needs my uvcdat pretty merged in and them your instructions so they can see the failure.

[doutriaux1]

Again we don't know if the failure prevents plot or if something else in diagnosticsWidgets.py prevents it once we are pas the failure

[mcenerney1]

To see the failure do the following. Assuming the data directories are in cam35_data and taylor_obs_data, bring up uvcdat. Under Diagnostics select AMWG. A new window comes up. Under Use Dataset 1, click Set Path and choose the cam35_data directory. Next select Use Observations 1. Set Path to taylor_obs_data. Make sure NCEP apears in the adjoining window. If not, select it in the menu. In the upper window that displays the complete list of diagnostics, select 14 - Taylor diagrams. Once done there is a section just below to set the variable options. In the Variables pull down menu, select T. In the seasons menu, select JAN. In the Variables Options menu, select "200 mbar". Now click Apply and off it goes. If you run into a snag, let me know.