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

generate range and pattern broken from GUI #1573

Closed
dnadeau4 opened this issue Sep 29, 2015 · 7 comments
Closed

generate range and pattern broken from GUI #1573

dnadeau4 opened this issue Sep 29, 2015 · 7 comments
Assignees
Milestone

Comments

@dnadeau4
Copy link
Contributor

@dnadeau4 dnadeau4 commented Sep 29, 2015

generating range 24 to 36 n = 6 ext on misses one color when geenrated

pattern is never generated
manually applying pattern is not picked up

@chaosphere2112
Copy link
Contributor

@chaosphere2112 chaosphere2112 commented Sep 30, 2015

@dnadeau4 First half is solved in the PR I just submitted, second half is documented further in #1577

Loading

@dnadeau4
Copy link
Contributor Author

@dnadeau4 dnadeau4 commented Sep 30, 2015

@chaosphere2112 thanks! It is hard to detect these kind of bugs. The software did not crash or anything, just did not answer back with patterns, but with filled colors.

Loading

@chaosphere2112
Copy link
Contributor

@chaosphere2112 chaosphere2112 commented Sep 30, 2015

Yup, took a bit of nosing around to determine where to point the finger.

Loading

@aashish24
Copy link
Contributor

@aashish24 aashish24 commented Oct 15, 2015

@chaosphere2112 should we close this issue?

Loading

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Oct 16, 2015

@chaosphere2112 I think you fixed this no?

Loading

@chaosphere2112
Copy link
Contributor

@chaosphere2112 chaosphere2112 commented Oct 16, 2015

Hasn't been merged yet (CDAT/VisTrails#59), but yeah, it's fixed.

Loading

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Oct 16, 2015

merged now.

Loading

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