-
Notifications
You must be signed in to change notification settings - Fork 440
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
Add GeoVista integration test #4568
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4568 +/- ##
=======================================
Coverage 95.77% 95.77%
=======================================
Files 127 127
Lines 21357 21357
=======================================
Hits 20454 20454
Misses 903 903 |
@tkoyama010 Hmmmm the Never seen that before, you? It almost seems like the test report isn't being sent to |
@bjlittle I don't understand why, but waiting is often the answer :) |
No need to worry above coverage here |
As this is CI PR, it is considered to be outside the 24-hour rule. Thank you all for your cooperation. |
@tkoyama010 @banesullivan @akaszynski Awesome, thanks so much guys for banking this PR! 🍻 Looking forward to meeting you all at SciPy! |
@bjlittle Thank you. Let's go drinking during SciPy. |
Overview
We should add GeoVista to our integration tests to make sure we account for how it uses PyVista when changing our API.
Ping @bjlittle
relate to #3967
resolves #4565
Details