From a8bf64c1f086772607d3cff3cb35ecdc1430d4c4 Mon Sep 17 00:00:00 2001 From: Luca Heltai Date: Fri, 10 May 2019 10:56:19 +0200 Subject: [PATCH] Documentation fixes. --- include/deal.II/fe/mapping.h | 5 ++++- include/deal.II/grid/tria_accessor.h | 5 +++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/include/deal.II/fe/mapping.h b/include/deal.II/fe/mapping.h index fb323e941f..acf799a63f 100644 --- a/include/deal.II/fe/mapping.h +++ b/include/deal.II/fe/mapping.h @@ -375,7 +375,10 @@ public: * MappingQEulerian, MappingQ1Eulerian, or MappingFEField. * * This function returns the bounding box containing all the vertices of the - * cell, as returned by the get_vertices() method. + * cell, as returned by the get_vertices() method. Beware of the fact that + * for higher order mappings this bounding box is only an approximation of the + * true bounding box, since it does not take into account curved faces, and it + * may be smaller than the true bounding box. * * @param[in] cell The cell for which you want to compute the center * diff --git a/include/deal.II/grid/tria_accessor.h b/include/deal.II/grid/tria_accessor.h index 47bd239917..b74b059dc0 100644 --- a/include/deal.II/grid/tria_accessor.h +++ b/include/deal.II/grid/tria_accessor.h @@ -1430,6 +1430,11 @@ public: /** * Return the smallest bounding box that encloses the object. + * + * Notice that this method is not aware of any mapping you may be using to + * do your computations. If you are using a mapping object that modifies the + * position of the vertices, like MappingQEulerian, or MappingFEField, then + * you should call the function Mapping::get_bounding_box() instead. */ BoundingBox bounding_box() const; -- 2.39.5