ffcx icon indicating copy to clipboard operation
ffcx copied to clipboard

Add support for mixed-dimensional kernels (codimension 1)

Open jpdean opened this issue 11 months ago • 1 comments

This PR adds support for mixed-dimensional kernels of codimension 1. It requires this PR to work.

It would be nice to avoid passing cell to analyse_modified_terminal. Please let me know if anyone has any thoughts on this.

jpdean avatar Mar 28 '24 21:03 jpdean

I think you've correctly noted that passing cell into analyse_modified_terminal is weird. It feels odd to store codim (which is the property of the element) to modified terminal, which has the purpose of defining the context in which the element/terminal is tabulated. In many places you already have the cell and the element, from which you can deduce the codimension, no?

For the analysis, I'd try to deduce the codim from cell and element and for the access you might need to store the codim as property of tabledata. Does that make sense or I am missing something?

michalhabera avatar Apr 04 '24 12:04 michalhabera

Thanks for the feedback everyone! This PR means that permutations are now only needed for the codim 0 cell, which simplifies this PR quite a lot. I think this will address most of the comments here, but I'll work on fixing the others soon.

jpdean avatar May 17 '24 17:05 jpdean

Are there any further issues anyone would like me to address?

jpdean avatar Jun 03 '24 08:06 jpdean