-
Notifications
You must be signed in to change notification settings - Fork 100
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
Trixi2Vtk test results for P4estMesh
on macOS are unverified
#681
Comments
I don't have access to Mac OS, so I can't do anything here... @sloede? |
Similar here. I asked @efaulhaber to create this issue such that we not forget about it. In general, I think we need a more robust way to check the results of Trixi2Vtk (xref #680), e.g., by writing to ASCII VTK and verifying that the contents match. The fact the macOS creates different hashes could be the result of something completely unrelated to VTK, e.g., because the used |
@andrewwinters5000 just converted a file on macOS for me. Interestingly, this file has the same hash as the files generated in CI by Linux and Windows. Any ideas why macOS in CI generates a different file? |
No idea, no. The only thing I can think of is something like differing One thing you could try is to write the VTK files without compression and/or as ASCII and then compare to see if there are actual differences in the files and from where: https://github.com/jipolanco/WriteVTK.jl#additional-options |
We will compare the VTK files directly using some tolerance, correct? |
Yes, that's the plan! |
The generated VTK files in the macOS CI tests of Trixi2Vtk (only the
P4estMesh
tests) have different hashes than on Linux and Windows. I made the tests pass like this, but it needs to be verified that the results are okay.The elixirs that need to be tested are
examples/p4est_2d_dgsem/elixir_euler_source_terms_nonperiodic.jl
andexamples/p4est_3d_dgsem/elixir_advection_amr_unstructured_curved.jl
. For both of them, the solution att=0.0
needs to be converted with Trixi2Vtk and compared to the results of another OS.The text was updated successfully, but these errors were encountered: