FEniCS / FEniCS/ffcx

CellDiameter not defined for higher-order cells

Open
#162 6 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
192
Forks
45
Avg merge
1d 11h
Merged PRs (30d)
16

Description

When using P2 geometry, `CellDiameter` raises a warning in UFL followed by an error in ffc:
```
WARNING:UFL:Only know how to compute cell diameter of P1 or Q1 cell.

".../access.py", line 66, in get
raise RuntimeError("Not handled: {}".format(type(e)))
RuntimeError: Not handled:
```

It is useful to have an estimate of the "cell diameter", even if it is not the actual length of an edge, but just the distance between vertices.

Should this be calculated in ffc, or ufl? Should it still be called `CellDiameter`?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.