Skip to content

Fix rc1 tests#1159

Merged
doutriaux1 merged 4 commits intomasterfrom
fix_rc1_tests
Mar 25, 2015
Merged

Fix rc1 tests#1159
doutriaux1 merged 4 commits intomasterfrom
fix_rc1_tests

Conversation

@doutriaux1
Copy link
Copy Markdown
Contributor

@aashish24 @chaosphere2112 @ThomasMaxwell this fixes my ubuntu ctests, goes with
CDAT/uvcdat-testdata#27

Both tests trying to write and read a test file with he same name
but the threshold is 12.4
adding 3 will le tit pass the test
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ThomasMaxwell that does not seem to be picked up by test suite

@doutriaux1
Copy link
Copy Markdown
Contributor Author

all right "qui ne dit mot consent". Merging.

doutriaux1 added a commit that referenced this pull request Mar 25, 2015
@doutriaux1 doutriaux1 merged commit a8e12d1 into master Mar 25, 2015
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

disabled the vector plots?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes since they do not work and are currently slightly different between mac and linux.

see: #653

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's only the proejction vectors plot that are disabled for test, regular proj tests for vectors are still in

@doutriaux1 doutriaux1 deleted the fix_rc1_tests branch May 14, 2015 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants