inducer / arraycontext

Choose your favorite numpy-workalike!
6 stars 11 forks source link

einsum: demote warning to comment #132

Closed matthiasdiener closed 2 years ago

matthiasdiener commented 2 years ago

This warning creates a lot of noise when running mirgecom

kaushikcfd commented 2 years ago

Now that the Loopy code-generator is smarter in pytato, we could make use of: https://documen.tician.de/pytato/array.html#pytato.tags.PrefixNamed.

inducer commented 2 years ago

Now that the Loopy code-generator is smarter in pytato, we could make use of: https://documen.tician.de/pytato/array.html#pytato.tags.PrefixNamed.

I agree that that's better. Implemented in #135, which I think supersedes this. That way, if einsum arguments end up materialized (which they often might be), these names won't be lost.