From 8631833c6c23b8ff3e173cd640f8445ae4a6ca1e Mon Sep 17 00:00:00 2001 From: bangerth Date: Wed, 27 Sep 2006 16:48:33 +0000 Subject: [PATCH] Add a TODO. git-svn-id: https://svn.dealii.org/trunk@13969 0785d39b-7218-0410-832d-ea1e28bc413d --- deal.II/deal.II/source/fe/fe_q.cc | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/deal.II/deal.II/source/fe/fe_q.cc b/deal.II/deal.II/source/fe/fe_q.cc index 2338673e4f..1c565d2d1a 100644 --- a/deal.II/deal.II/source/fe/fe_q.cc +++ b/deal.II/deal.II/source/fe/fe_q.cc @@ -21,6 +21,15 @@ #include + +//TODO: we should have a copy constructor for this class. In hp methods, we often do things like +// fe_collection.push_back(FE_Q(7)) +// which becomes really expensive in 3d because we here have to create all the embedding and +// other matrices twice. for example, hp/hp_hanging_nodes_02 spends about 70% of its total +// running time in FullMatrix::Tmmult, which is only called from the matrix creation functions +// of this class... + + // namespace for some functions that are used in this file. they are // specific to numbering conventions used for the FE_Q element, and // are thus not very interesting to the outside world. we'd like to -- 2.39.5