-
Notifications
You must be signed in to change notification settings - Fork 93
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
element -> cell in transformation functions #1124
Comments
I am fine with this. We should in this turn also include a table similar to https://defelement.org/ciarlet.html#Cell+subentities somewhere (with credits to this page ofc). Furthermore, I think we should clarify what we mean with "element" (and maybe "finite element"), since I think I introduced the work "element" into Ferrite. |
We have included that information here: https://ferrite-fem.github.io/Ferrite.jl/dev/topics/reference_shapes/#Entity-naming
That sounds good to me. Can try to write something up on that when I find time unless someone beats me to it. |
Okay, maybe we should rename the section title. I am not sure that users will expect to find the information in htere https://ferrite-fem.github.io/Ferrite.jl/dev/topics/reference_shapes/#Entity-naming |
Any suggestions? |
Before doing any writing of this, the definitions that I have in my head would be something like the following. (Please give feedback)
|
To be consistent with the notation in Ferrite,
facet_to_element_transformation
->facet_to_cell_transformation
element_to_facet_transformation
->cell_to_facet_transformation
(edited to as I mistakenly used cell on both from and to...)
Ferrite.jl/src/FEValues/face_integrals.jl
Lines 1 to 15 in b8f986a
Since this is only in devdocs, I guess we can change it as a minor version change?
For the documentation, we are also mixing (discussed somewhere else before, but to keep the discussion together, pointing out here too) between cell, element, and (reference) shape. From our convention in #914, I think we ideally should have
ReferenceCell
and notReferenceShape
, but sinceAbstractReferenceShape
already exists, I guess it makes sense to keep shape for the reference (cells/shapes)? However, I think removing theelement
from https://ferrite-fem.github.io/Ferrite.jl/dev/devdocs/elements/ would make sense.Thoughts @termi-official, @lijas, @fredrikekre, and more?
The text was updated successfully, but these errors were encountered: