update(modpath example): update and remove deprecated code from pyvista VTK example#153
Conversation
|
Hoping that these changes take care of the VTK failure issue on windows. If it doesn't, I'll try to investigate further into what might be causing the failure. |
|
@jlarsen-usgs - looks like specifying OpenGL at 1.30 is causing issues on Windows. https://github.com/DOI-USGS/python-for-hydrology/actions/runs/15142496327/job/42571099793?pr=153 |
|
@mnfienen -- Still trying to figure out how to fix this. There's not a lot of information about patching this error that I can find |
|
right on - maybe we just exclude this nb from testing? It's the one that often has dependency issues.... |
|
Excluding the test from windows might be the easiest course of action. I sent @wpbonelli a message to see if he's seen this error in the flopy testing before, because it runs the same example with pyvista |
|
right on - let's see if @wpbonelli has a fix and if not, I guess we can exclude. Thanks for looking into this! |
|
hey @jlarsen-usgs - I was waiting to see if you got any clarity from Wes. If not, I guess we can exclude the test for now as you suggest. |
|
Sorry guys I lost track of this. I don't know how to fix this yet. We are seeing it on flopy ci too. |
|
@jlarsen-usgs and @wpbonelli - any chance we can revisit this again? Maybe just remove Windows for now? |
|
@mnfienen @jlarsen-usgs I found pyvista/pyvista#4104 and beat my head on this on the flopy side for some time (environment variable magic, trying to get mesa3d instead, etc) and did not prevail so my rec would be to skip this one on windows for now |
|
@mnfienen updated testing to skip the modpath notebook on windows |
|
I just realized we do this in the mf6 examples ci with https://github.com/pyvista/setup-headless-display-action - name: Set up headless display
uses: pyvista/setup-headless-display-action@v4maybe worth a shot. I can try it out on flopy and report back. |
|
thanks @jlarsen-usgs and @wpbonelli for sorting this out! |
No description provided.