Skip to content
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

Group integrals by metadata #70

Open
IgorBaratta opened this issue Oct 25, 2021 · 2 comments
Open

Group integrals by metadata #70

IgorBaratta opened this issue Oct 25, 2021 · 2 comments
Labels

Comments

@IgorBaratta
Copy link
Member

Currently UFL groups integrals by domain, integral_type, subdomain_id into IntegralData objects.
The form compiler then uses this to generate different kernels.

However, forms with different metadata are grouped, and in FFCx, this can potentially produce an error (see: #Issue 394).

Ideally, we would like to have one kernel by metadata (and possibly a quadrature rule per kernel).
Our proposal is to modify build_integral_data, to group integrals by metadata.

@wence- Any thoughts on that? Would it be ok for tsfc?

@wence-
Copy link
Collaborator

wence- commented Oct 25, 2021

We'd prefer if the integral were not split by metadata, TSFC would continue to work but would produce slightly worse code. You could augment compute_form_data with yet another flag that defaults to false split_integrals_by_metadata=False or something?

@IgorBaratta
Copy link
Member Author

Yes. I think another flag in compute_form_data would do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants