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

Vtk ui test coverage #1426

Merged
merged 29 commits into from Aug 5, 2015
Merged

Vtk ui test coverage #1426

merged 29 commits into from Aug 5, 2015

Conversation

@chaosphere2112
Copy link
Contributor

@chaosphere2112 chaosphere2112 commented Jun 25, 2015

Fully rebased PR that replaces #1304 to make this all nice and mergeable.

@chaosphere2112
Copy link
Contributor Author

@chaosphere2112 chaosphere2112 commented Jun 25, 2015

@chaosphere2112
Copy link
Contributor Author

@chaosphere2112 chaosphere2112 commented Jun 25, 2015

@jbeezley or @aashish24 Can someone kick the build off on garant? I accidentally submitted this PR before the one on uvcdat-testdata (my remotes weren't what I thought they were).

@chaosphere2112
Copy link
Contributor Author

@chaosphere2112 chaosphere2112 commented Jul 27, 2015

@aashish24 @jbeezley @doutriaux1 This branch is ready to merge.

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Aug 4, 2015

hum... I rebased with master into this and tried. I get this:
https://open.cdash.org/viewTest.php?onlyfailed&buildid=3937105
Not sure why.

I'm worried about the matlplotlib. I need to test again master....

doutriaux1 added a commit that referenced this issue Aug 5, 2015
@doutriaux1 doutriaux1 merged commit d137f6e into master Aug 5, 2015
3 checks passed
@doutriaux1 doutriaux1 deleted the vtk_ui_test_coverage branch Aug 5, 2015
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