Skip to content

L2ProjectionInverseDiscretizationConnection should allow modal output #206

@inducer

Description

@inducer

This could avoid the forced (and possibly un-needed) modal-to-nodal conversion at the end of the connection:

return DOFArray(
actx,
data=tuple(
actx.einsum("ij,ej->ei",
vandermonde_matrix(grp),
c_i,
arg_names=("vdm", "coeffs"),
tagged=(FirstAxisIsElementsTag(),))
for grp, c_i in zip(self.to_discr.groups, coefficients)
)
)

I think this is easy to do since it receives a connection as input. It just needs to check whether the output side of the connection is modal, and if so, skip the extra conversion to nodal.

I noticed this while reviewing #192.

cc @alexfikl @thomasgibson

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions