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

Fixes bug where hovering over meshfill raised exceptions #1769

Merged
merged 1 commit into from Jan 7, 2016
Merged

Conversation

@chaosphere2112
Copy link
Contributor

@chaosphere2112 chaosphere2112 commented Jan 7, 2016

Fixes #1764 by adding a missing try/except

@aashish24
Copy link
Contributor

@aashish24 aashish24 commented Jan 7, 2016

@chaosphere2112 why back would not be a vtkProp?

@aashish24
Copy link
Contributor

@aashish24 aashish24 commented Jan 7, 2016

Other than that LGTM 👍

@chaosphere2112
Copy link
Contributor Author

@chaosphere2112 chaosphere2112 commented Jan 7, 2016

Meshfill apparently stores a string in the display plot's backend; a pretty diverse list of objects get shoved in there, not exclusively vtkProps. I had added try/except checks for that call in two other places, just missed it for this one.

@aashish24
Copy link
Contributor

@aashish24 aashish24 commented Jan 7, 2016

aashish24 added a commit that referenced this issue Jan 7, 2016
Fixes bug where hovering over meshfill raised exceptions
@aashish24 aashish24 merged commit 11ceb27 into master Jan 7, 2016
4 of 9 checks passed
@aashish24 aashish24 deleted the fix_1764 branch Jan 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants