Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error while plotting meshfill plot #1284

Closed
vvpalav opened this issue May 6, 2015 · 10 comments
Closed

Error while plotting meshfill plot #1284

vvpalav opened this issue May 6, 2015 · 10 comments

Comments

@vvpalav
Copy link

@vvpalav vvpalav commented May 6, 2015

Exception is caught while plotting mesh fill plot. Entire application crashes after this.

Traceback (most recent call last):
  File "/Users/vinayakpalav/Documents/uvcdat_2_2/15-05-05/vistrails/vistrails/core/modules/vistrails_module.py", line 327, in update
    self.compute()
  File "/Users/vinayakpalav/Documents/uvcdat_2_2/15-05-05/vistrails/vistrails/packages/uvcdat_cdms/init.py", line 1276, in compute
    self.cellWidget = self.displayAndWait(QCDATWidget, input_ports)
  File "/Users/vinayakpalav/Documents/uvcdat_2_2/15-05-05/vistrails/vistrails/packages/spreadsheet/basic_widgets.py", line 247, in displayAndWait
    return spreadsheetWindow.displayCellEvent(e)
  File "/Users/vinayakpalav/Documents/uvcdat_2_2/15-05-05/vistrails/vistrails/packages/spreadsheet/spreadsheet_window.py", line 490, in displayCellEvent
    sheet.setCellByType(row, col, e.cellType, e.inputPorts)
  File "/Users/vinayakpalav/Documents/uvcdat_2_2/15-05-05/vistrails/vistrails/packages/spreadsheet/spreadsheet_tab.py", line 320, in setCellByType
    newCell.updateContents(inputPorts)
  File "/Users/vinayakpalav/Documents/uvcdat_2_2/15-05-05/vistrails/vistrails/packages/uvcdat_cdms/init.py", line 1450, in updateContents
    self.canvas.plot(cgm,*args,**kwargs)
  File "/Users/vinayakpalav/Documents/uvcdat_2_2/15-05-05/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/vcs/Canvas.py", line 2617, in plot
    a = self.__plot( arglist, keyargs )
  File "/Users/vinayakpalav/Documents/uvcdat_2_2/15-05-05/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/vcs/Canvas.py", line 2730, in __plot
    arglist[1]=MV2.array(g.getMesh())
UnboundLocalError: local variable 'g' referenced before assignment

GUI:

screen shot 2015-05-05 at 11 43 42 pm

@remram44
Copy link
Contributor

@remram44 remram44 commented May 6, 2015

No, I don't see this error

resized_taskmngr-launcher1

@vvpalav
Copy link
Author

@vvpalav vvpalav commented May 6, 2015

Just tried again, same exception. FYI - I have a fresh build of UVCDAT from master.
@aashish24 @doutriaux1 can you please try this one.

@remram44
Copy link
Contributor

@remram44 remram44 commented May 6, 2015

You built the wrong branch, release has the fix (#1247)

@remram44 remram44 closed this May 6, 2015
@vvpalav
Copy link
Author

@vvpalav vvpalav commented May 6, 2015

👍 .. I built from master

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented May 6, 2015

@aashish24 we should really merge release PR into master as well.

@remram44
Copy link
Contributor

@remram44 remram44 commented May 6, 2015

Release branches are usually merged into development... Git really intends for 'master' to be the latest.

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented May 6, 2015

I agree.

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented May 6, 2015

@aashish24 should I go "dirty" and merge release into master? Or do you (or anyone else) want to take the time to do this cleanly.

@remram44
Copy link
Contributor

@remram44 remram44 commented May 6, 2015

If you do, there'll be exactly no difference between release and master... I don't know why there are two branches to begin with.

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented May 6, 2015

things are going to go into master soon that shouldn't go into release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants