-
Notifications
You must be signed in to change notification settings - Fork 145
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
Issue488 gmsh meshing changed #573
Conversation
Apparently when written, Gmsh didn't produce the expected grid, but now it does (although still not precisely at the spacing requested)
MethodSince this contribution changes FiPy's behavior depending on the Gmsh version, $ git checkout develop
$ for d in ~/repositories/gmsh/gmsh-*
> do
> cp $d/bin/gmsh /users/tnk10/.conda/envs/fipy/bin/gmsh
> python setup.py test &> test_develop_$(basename $d).log
> done
$ git checkout issue488_gmsh-meshing-changed
$ for d in ~/repositories/gmsh/gmsh-*
> do
> cp $d/bin/gmsh /users/tnk10/.conda/envs/fipy/bin/gmsh
> python setup.py test &> test_issue488_$(basename $d).log
> done ResultsAt a minimum, this contribution should not adversely affect the test results. $ for f in test_develop_gmsh-*.log
> do
> diff -y --suppress-common-lines $f ${f/develop/issue488} > diff_${f/develop_/}
> done Gmsh v2.7.0 (2013-03-08)Passed: fewer failures
Gmsh v2.7.1 (2013-05-12)Passed: fewer failures
Gmsh v2.8.0 (2013-07-09)Passed: fewer failures
Gmsh v2.11.0 (2015-11-07)Passed: fewer failures
Gmsh v3.0.6 (2017-11-05)Passed: fewer failures
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Binary distributions of Gmsh prior to 2.7.0 were not available in 64-bit variants, so I am unable to test whether those versions properly work. However, since the five recent versions I have tested produced fewer errors, these changes look good to me.
Fixes #488 |
Only apply mesh epsilon for older versions of gmsh
Fixes #488