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

WIP: Increase minimum julia version to v1.10 #101

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bennibolm
Copy link
Contributor

No description provided.

test/test_trixi2vtk.jl Outdated Show resolved Hide resolved
@sloede
Copy link
Member

sloede commented Dec 20, 2024

For some reason, the p4est tests fail. Do they just need updated reference files or what's the deal here?

@bennibolm
Copy link
Contributor Author

For some reason, the p4est tests fail. Do they just need updated reference files or what's the deal here?

I was wondering about that as well.
First, I wasn't even sure, if it would work to just set the commit hash to the commit in my branch as I did in the last commit. Still not fully sure, if it should work 😅 So, I still have some hope that the errors disappear when the reference files are updated.
Secondly, @benegee also got similar result in the T8codeMesh-Trixi2Vtk PR. His Windows tests also work, but Ubuntu and macOS don't. There, we were very confused that some tests even work at all, since there are no T8codeMesh files in the reference file repository yet.
It seems to have something to do with the function get_reference_file and that it just uses some sort of local files when the file path doesn't exists(?). Please correct me if I got that wrong.

@sloede
Copy link
Member

sloede commented Dec 20, 2024

It seems to have something to do with the function get_reference_file and that it just uses some sort of local files when the file path doesn't exists(?).

By default it uses a file it finds locally, such that multiple tests using the same input file do not have to download it over and over again. A quick test if the old files somehow "pollute" the tests is by setting the default value of the force kwarg to true, such that each reference file must be downloaded anew each time it is used. If this fixes the issues, it would be a strong indicator that some caching is going wrong and we can fix it.

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.

2 participants