Skip to content

Create v1.4.0 release #2029

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

Closed
jwpeterson opened this issue Feb 10, 2019 · 1 comment
Closed

Create v1.4.0 release #2029

jwpeterson opened this issue Feb 10, 2019 · 1 comment

Comments

@jwpeterson
Copy link
Member

Reference this ticket in bug fixes, etc. that get cherry-picked over from master onto the release branch.

jwpeterson added a commit that referenced this issue Feb 10, 2019
jwpeterson added a commit that referenced this issue Feb 10, 2019
jwpeterson added a commit that referenced this issue Feb 10, 2019
jwpeterson added a commit that referenced this issue Feb 11, 2019
The Elem::set_p_level() API is only available when LIBMESH_ENABLE_AMR,
since it is a feature used for p-refinement.

Refs #2029.
jwpeterson added a commit that referenced this issue Feb 11, 2019
LIBMESH_CHKERR must be called from a ParallelObject when exceptions
are not enabled.  That is because, in that specific case, the
LIBMESH_CHKERR macro just forwards on to the old CHKERRABORT macro,
which requires a communicator.

Refs #2029.
jwpeterson added a commit that referenced this issue Feb 11, 2019
Therefore we should not have a unit test with a std::vector<Node>.

Refs #2029.
jwpeterson pushed a commit that referenced this issue Feb 11, 2019
jwpeterson added a commit that referenced this issue Feb 16, 2019
I think because of the update to automake 1.16.1, build-aux/config.guess
is now reporting that my system is "x86_64-pc-linux-gnu", so I needed to
add a new symbolic for the Tecplot binaries. Note that you have to
force add (add -f) these links because otherwise they are ignored by
the "lib" line in .gitignore.

Refs #2029.
jwpeterson added a commit that referenced this issue Feb 16, 2019
I thought about having "make distcheck" automatically
initialize/update the metaphysicl submodule, but that's way too
complicated/too many things can go wrong so I settled on just throwing
a human-readable error if the user runs "make dist" or "make
distcheck" without first initializing the submodule.

Refs #2029.
jwpeterson added a commit that referenced this issue Feb 16, 2019
jwpeterson added a commit that referenced this issue Feb 16, 2019
The basic idea is that we don't want to mix building our own
METIS and using PETSc's ParMETIS. The different possibilities are:

                       | Parmetis in PETSc?
                       |   yes  |    no
                   ----------------------
                   yes |    A   |    B
METIS in PETSc?    ----------------------
                   no  |    C   |    D

A: PETSc was built with both METIS and ParMETIS support. In this case,
   we use both from PETSc.
B: PETSc is built with METIS support but no ParMETIS support. In this
   case, we use the METIS support and disable ParMETIS.
C: PETSc is built with no METIS support, but does have ParMETIS. I'm not
   sure this is really possible, but in this case, we would build our
   own METIS and disable ParMETIS support just to avoid mixing them.
D: PETSc is built with neither METIS nor ParMETIS support. In this case,
   we build both on our own.

Refs #2029.
jwpeterson added a commit that referenced this issue Feb 16, 2019
This doesn't change any code, it just updates some of the bootstrapped
files to use the same version of automake as libmesh so there are no
diffs when we run bootstrap at the libmesh level.

Refs #2029.
jwpeterson added a commit that referenced this issue Feb 16, 2019
jwpeterson added a commit to jwpeterson/libmesh that referenced this issue Feb 21, 2019
There was an issue with subdomain restricted variables. I previously
did not realize that they would still show up in the discontinuous
solution vector (as zero padding) so I was not indexing correctly in
that case. This fix should also be cherry-picked onto v1.4.0 since
this API is present in that version.

Refs libMesh#2029.
Refs libMesh#2010.
jwpeterson added a commit to jwpeterson/libmesh that referenced this issue Feb 21, 2019
Instead of setting TAR_OPTIONS, I also tried calling
AC_SUBST(am__tar,[tar --hard-dereference]) but unforunately this
alternate approach did not work, perhaps the AC_SUBST comes too late
in the process to take effect or perhaps it is simply not possible to
do this. It was an off-hand remark in a mailing list [0], not an
officially sanctioned way of doing things.

[0]: http://gnu-automake.7480.n7.nabble.com/bug-19616-dist-tarball-contains-hardlinks-td21667.html

Refs libMesh#2046.
Refs libMesh#2029.
jwpeterson added a commit that referenced this issue Feb 21, 2019
There was an issue with subdomain restricted variables. I previously
did not realize that they would still show up in the discontinuous
solution vector (as zero padding) so I was not indexing correctly in
that case. This fix should also be cherry-picked onto v1.4.0 since
this API is present in that version.

Refs #2029.
Refs #2010.
jwpeterson added a commit that referenced this issue Feb 22, 2019
Instead of setting TAR_OPTIONS, I also tried calling
AC_SUBST(am__tar,[tar --hard-dereference]) but unforunately this
alternate approach did not work, perhaps the AC_SUBST comes too late
in the process to take effect or perhaps it is simply not possible to
do this. It was an off-hand remark in a mailing list [0], not an
officially sanctioned way of doing things.

[0]: http://gnu-automake.7480.n7.nabble.com/bug-19616-dist-tarball-contains-hardlinks-td21667.html

Refs #2046.
Refs #2029.
jwpeterson pushed a commit that referenced this issue Feb 26, 2019
Currently if not LIBMESH_HAVE_CXX14_MAKE_UNIQUE we define libmesh_make_unique
to be equivalent to make_unique without namespace scoping. I believe
explicitly scoping to libMesh::make_unique makes sense

Refs #2029.
jwpeterson pushed a commit that referenced this issue Feb 27, 2019
jwpeterson added a commit that referenced this issue Mar 1, 2019
jwpeterson added a commit that referenced this issue Mar 1, 2019
@jwpeterson
Copy link
Member Author

1.4.0 was released on March 11, 2019. There have been a couple of small bug fixes since then, so I'm contemplating a 1.4.1 release at some point if enough of them accumulate. I'll keep using this issue number for those bug fixes.

jwpeterson pushed a commit that referenced this issue Mar 15, 2019
Really we should be rejecting 1.16 too now but that's a less trivial
fix

Refs #2029.
jwpeterson pushed a commit that referenced this issue Mar 15, 2019
Otherwise debug-mode PETSc builds (rightly) scream at us for starting
set operations without finishing add operations.

Refs #2029.
jwpeterson added a commit that referenced this issue Mar 15, 2019
jwpeterson added a commit that referenced this issue Mar 15, 2019
jwpeterson pushed a commit that referenced this issue Mar 15, 2019
jwpeterson added a commit that referenced this issue Mar 15, 2019
jwpeterson added a commit that referenced this issue Mar 15, 2019
Fixes one of the issues reported in #2071.

Note: our current CIVET complex testing is not running or compiling
the unit tests, so this slipped through the cracks. I have a different
PR in that will hopefully address the unit testing issue so this won't
happen again.

Refs #2029.
jwpeterson added a commit to jwpeterson/libmesh that referenced this issue Mar 18, 2019
This API was present in 1.4.0, so this commit should also be
cherry-picked onto the in-progress v1.4.1_branch.

Refs libMesh#2029.
jwpeterson added a commit that referenced this issue Mar 18, 2019
This API was present in 1.4.0, so this commit should also be
cherry-picked onto the in-progress v1.4.1_branch.

Refs #2029.
jwpeterson added a commit that referenced this issue Mar 26, 2019
jwpeterson added a commit that referenced this issue Mar 26, 2019
This function did not wind up making it into any PETSc release.  This
should probably also be cherry-picked onto the v1.4.1 branch. Even
though the likelihood of any tagged libmesh release continuing to work
with PETSc master is negligibly small, the tagged release _might_ work
with a future PETSc 3.11 so we should try and support that if
possible.

Refs #2084.
Refs #2029.
jwpeterson added a commit that referenced this issue Mar 28, 2019
This would have been found by "make installcheck" on a build with
CapnProto, SLEPC, and GLPK enabled, if we ever ran such a thing...
This commit will also be cherry-picked onto the 1.4.1 branch.

Refs #2029.
jwpeterson pushed a commit that referenced this issue Apr 9, 2019
We've never been correctly copying remote_elem neighbor links in the
skip_find_neighbors code path (or rather, we've been copying them
correctly and then accidentally *undoing* them), and since #2028
made skip_find_neighbors the default behavior in more code paths, that
bug was starting to be triggered e.g. in the #1938 tests.

This commit gets remote_elem into the old-to-new-elements map so the
skip_find_neighbors code path will recognize it, and removes a bit of
redundancy in the rest of the old-to-new-elements map initialization,
and moves the manual remote_elem-specific copying code into the
!skip_find_neighbors path to avoid redundancy with that.

Refs #2029.
jwpeterson pushed a commit that referenced this issue Apr 10, 2019
This was part of a half-baked attempt to support both "refine and
coarsen prior to EquationSystems::reinit for CPU efficiency" and
"refine and coarsen only within EquationSystems::reinit for memory
efficiency" idioms, but the latter support was seriously broken.

Refs #2029.
jwpeterson pushed a commit that referenced this issue Apr 10, 2019
This is a little more readable

Backported with the other commits from #2099 with the command.
git cherry-pick e9297f1 33466d0 ae5cbc1 de07ffc

Refs #2029.
jwpeterson pushed a commit that referenced this issue Apr 10, 2019
This catches a bug much earlier in runtime: when bad constraint
equations are created, rather than when they are used.

Backported with the other commits from #2099 with the command.
git cherry-pick e9297f1 33466d0 ae5cbc1 de07ffc

Refs #2029.
jwpeterson pushed a commit that referenced this issue Apr 10, 2019
This turned out to be a red herring in my search for a different bug,
but it's not a bad thing to keep double-checking.

Backported with the other commits from #2099 with the command.
git cherry-pick e9297f1 33466d0 ae5cbc1 de07ffc

Refs #2029.
jwpeterson pushed a commit that referenced this issue Apr 10, 2019
If we have a subdomain-restricted variable which is supported on a
fine element but *not* supported on a neighboring coarse element, then
we do *not* want to try to generate hanging node constraints for that
interface.  Doing so was giving us OOB reads and garbage DoF IDs.

Backported with the other commits from #2099 with the command.
git cherry-pick e9297f1 33466d0 ae5cbc1 de07ffc

Refs #2029.
jwpeterson added a commit that referenced this issue Apr 18, 2019
We only set h_min in this way if it has not been previously
updated. This ensures that h_min is the min over both meshes being
stitched.

Refs #2029.

Co-authored-by: John W. Peterson <[email protected]>
jwpeterson added a commit that referenced this issue Apr 30, 2019
jwpeterson pushed a commit that referenced this issue May 2, 2019
Otherwise we'll happily set inconsistent flags and not discover the
error until we actually start trying mesh refinement.

Refs #2029.
jwpeterson pushed a commit that referenced this issue May 2, 2019
Now that we're asserting error indicator sanity, we need to be sure
that our linear solvers really converge even when no special
LIBMESH_OPTIONS are used.

Refs #2029.
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

No branches or pull requests

1 participant