Rewrite FEValues/FiniteElement/Mapping interfaces #1198

Closed
bangerth opened this Issue Jul 26, 2015 · 5 comments

Projects

None yet

3 participants

@bangerth
Member

Fixing #1152, I realized again that I do not understand (and have never really understood) the way FEValues, FiniteElement and Mapping classes interact. Looking at how all of this works I find that much isn't well documented, some of the interfaces could be improved, etc.

This issue is simply a catch-all for all of the patches I'm doing to improve the situation.

@guidokanschat
Contributor

I do not believe we should invest heavily in FEValues anymore. It is inherently memory bound and does not support tensor product integration like it is used in MatrixFree. Let's rather focus on replacing it and while we are doing so, separate Mapping in a way consistent with manifold descriptions.

@bangerth
Member

Fortunately, ours is a project big enough so that different people can work on different parts :-) (I'm not implementing new functionality, just cleaning up interfaces.)

@bangerth bangerth changed the title from Rewrite FEValues/FiniteElement/Mapping interfaces to [Meta] Rewrite FEValues/FiniteElement/Mapping interfaces Aug 4, 2015
This was referenced Aug 5, 2015
@tamiko tamiko added the Enhancement label Aug 25, 2015
@tamiko tamiko changed the title from [Meta] Rewrite FEValues/FiniteElement/Mapping interfaces to Rewrite FEValues/FiniteElement/Mapping interfaces Aug 25, 2015
@tamiko tamiko added the Meta label Aug 25, 2015
@tamiko
Member
tamiko commented Aug 25, 2015

I have no idea what a "Meta" issue should be. But now it is one.

@bangerth
Member

Meta-Issue: An issue that is a placeholder for a sequence of other ones. E.g., an issue that is referenced from every pull request or issue related to getting the next release out of the door.

@tamiko tamiko added this to the Release 8.4 milestone Aug 27, 2015
@bangerth bangerth added a commit to bangerth/dealii that referenced this issue Sep 2, 2015
@bangerth bangerth Deprecate FEValues::transform().
I don't see the use of this function given that all the
transformations from reference to real cell should actually go
through the Mapping classes directly. The function is also not
used anywhere.

Fixes #1509. In reference to #1198.
880cf88
@bangerth bangerth added a commit to bangerth/dealii that referenced this issue Sep 2, 2015
@bangerth bangerth Deprecate FEValues::transform().
I don't see the use of this function given that all the
transformations from reference to real cell should actually go
through the Mapping classes directly. The function is also not
used anywhere.

Fixes #1509. In reference to #1198.
e71fda6
This was referenced Sep 3, 2015
@bangerth
Member
bangerth commented Feb 9, 2016

This is as far as I intend to take it for now.

@bangerth bangerth closed this Feb 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment