Skip to content

Conversation

@thomasgibson
Copy link
Collaborator

Copy link
Owner

@inducer inducer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Comments from our meeting)


faces = faces_for_shape(base_grp.shape)
nfaces = len(faces)
# NOTE: assumes same quadrature rule on all faces
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code this as an assert.

Copy link
Owner

@inducer inducer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More comments from (the second half) of our meeting

Comment on lines 575 to 576
_reshape((1, -1), qtilde_allquad),
_reshape((-1, 1), qtilde_allquad),
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try actx.np.reshape.

Comment on lines 575 to 576
_reshape((1, -1), qtilde_allquad),
_reshape((-1, 1), qtilde_allquad),
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

(i.e. there's some benefit to doing this operator-ly)

Comment on lines 479 to 481
mass_flux = rho_mean * u_avg
momentum_flux = np.outer(mass_flux, u_avg) + np.eye(dim) * p_mean
energy_flux = (
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

_flux_div here?

Comment on lines +131 to +138
q_mats = [p_mat.T @ (weights * vdm_q.T @ vdm_q) @ diff_mat @ p_mat
for diff_mat in diff_matrices(vgrp)]
e_mat = vf_mat @ p_mat
qtilde_mats = 0.5 * np.asarray(
[np.block([[q_mats[d] - q_mats[d].T, e_mat.T @ b_mats[d]],
[-b_mats[d] @ e_mat, b_mats[d]]])
for d in range(dcoll.dim)]
)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not loving that this repeats code from the hybridized op implementation.

@thomasgibson thomasgibson added the mirgecom production Developments critical to MIRGECom label Jan 28, 2022
@thomasgibson thomasgibson marked this pull request as ready for review January 28, 2022 04:52
@inducer
Copy link
Owner

inducer commented Apr 14, 2023

See #306 for an initial attempt at updating this for modern-day grudge.

inducer added a commit that referenced this pull request Apr 14, 2023
This is #214 squashed and rebased.

Co-authored-by: Andreas Kloeckner <inform@tiker.net>
inducer added a commit that referenced this pull request Apr 14, 2023
This is #214 squashed and rebased.

Co-authored-by: Andreas Kloeckner <inform@tiker.net>
@inducer
Copy link
Owner

inducer commented Jun 23, 2023

Closing in favor of #306.

inducer added a commit that referenced this pull request Jun 23, 2023
This is #214 squashed and rebased.

Co-authored-by: Andreas Kloeckner <inform@tiker.net>
@inducer inducer closed this Jun 23, 2023
@inducer inducer deleted the flux-differencing branch June 23, 2023 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mirgecom production Developments critical to MIRGECom

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants