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

isolines GUI editor not connected #1625

Closed
doutriaux1 opened this issue Oct 21, 2015 · 9 comments
Closed

isolines GUI editor not connected #1625

doutriaux1 opened this issue Oct 21, 2015 · 9 comments
Assignees
Milestone

Comments

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Oct 21, 2015

No description provided.

@chaosphere2112
Copy link
Contributor

@chaosphere2112 chaosphere2112 commented Oct 21, 2015

Actually, it turns out isoline.linewidths and isoline.line aren't being obeyed. I'll still fix it, already found the right chunk of code.

@doutriaux1
Copy link
Contributor Author

@doutriaux1 doutriaux1 commented Nov 10, 2015

@chaosphere2112 this one is fixed now correct?

@aashish24
Copy link
Contributor

@aashish24 aashish24 commented Nov 10, 2015

@doutriaux1 I don't think we exposed the spacing attribute in the GUI @chaosphere2112 correct me if I am mistaken.

@doutriaux1
Copy link
Contributor Author

@doutriaux1 doutriaux1 commented Nov 10, 2015

@aashish24 it seems to be in
spacings

@chaosphere2112
Copy link
Contributor

@chaosphere2112 chaosphere2112 commented Nov 10, 2015

@doutriaux1 The specific issue that was referenced in this bug (which was that the fields that were present in the GUI didn't do anything) is addressed in #1627, which is on @sankhesh's plate (as it quickly left the depth of my VTK skills).

@aashish24
Copy link
Contributor

@aashish24 aashish24 commented Nov 10, 2015

@aashish24 it seems to be in

@doutriaux1 where do you see it? Spacings if for streamlines..

@sankhesh
Copy link
Contributor

@sankhesh sankhesh commented Nov 10, 2015

Yes. I confirm it is on my plate. I should be able to get to it today.

@aashish24
Copy link
Contributor

@aashish24 aashish24 commented Nov 13, 2015

I have created a new issue for this and assigned it to @chaosphere2112

@aashish24 aashish24 closed this Nov 13, 2015
@sankhesh
Copy link
Contributor

@sankhesh sankhesh commented Nov 13, 2015

This issue is not fixed until #1688 is merged.

@sankhesh sankhesh reopened this Nov 13, 2015
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
4 participants