gmarkall / manycore_form_compiler

MCFC is deprecated. See https://code.launchpad.net/~grm08/ffc/pyop2
https://code.launchpad.net/~grm08/ffc/pyop2
GNU General Public License v3.0
3 stars 1 forks source link

Coefficient evaluation hard coded for P1 #40

Closed kynan closed 12 years ago

kynan commented 13 years ago

Currently, the generated code for coefficient evaluation has a hard coded assumption of P1 elements:

gmarkall commented 13 years ago

Are you talking about the loop that gets the value of the coefficients at the quadrature points from its values at the nodes? Isn't there always a loop over quadrature points because it's computing the values at the quadrature points?

Also, where is the assumption that elements are P1? I think there's a loop over the basis functions (i_r_0 will be the induction variable in a lot of cases) whose upper bound will vary with the number of basis functions.

Is there another assumption somewhere?

dham commented 13 years ago

I don't think that's a P1 assumption. All element integrals can be conducted as a sum over quadrature points.

What is currently absent is a P0 optimisation because P0 coefficients can go outside the quadrature loop. However this is an OPTIMISATION and we really don't want to look at it until after we have correct code running, however slowly.