Title: Attempt to make `BcastUntilActxArray` work for broadcastees that are containers by majosm · Pull Request #313 · inducer/arraycontext · GitHub
Open Graph Title: Attempt to make `BcastUntilActxArray` work for broadcastees that are containers by majosm · Pull Request #313 · inducer/arraycontext
X Title: Attempt to make `BcastUntilActxArray` work for broadcastees that are containers by majosm · Pull Request #313 · inducer/arraycontext
Description: In some mirgecom examples, dt can be a DOFArray. When using _compiled_lsrk45_step from grudge (which now uses BcastUntilActxArray for h, etc.), this causes dt to broadcast down to the array context arrays of the RHS container, leading to the creation of erroneous nested DOFArrays. So it looks like we need to tweak BcastUntilActxArray to handle the case of broadcastee being an array container. I took a stab at doing so here. I don't know if this is the right approach, but my hunch is that we should let the container's defined broadcasting behavior take over in this case (mainly because I feel like, e.g., bcast(Container()) + Container() ought to be equivalent to Container() + Container() and not do any kind of broadcasting inside). Thoughts @inducer, @alexfikl?
Open Graph Description: In some mirgecom examples, dt can be a DOFArray. When using _compiled_lsrk45_step from grudge (which now uses BcastUntilActxArray for h, etc.), this causes dt to broadcast down to the array context...
X Description: In some mirgecom examples, dt can be a DOFArray. When using _compiled_lsrk45_step from grudge (which now uses BcastUntilActxArray for h, etc.), this causes dt to broadcast down to the array context...
Opengraph URL: https://github.com/inducer/arraycontext/pull/313
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:63fa8e3a-c41a-8c5d-a5b6-3c51be700ad4 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 83D6:11597D:1009AFB:16B50D8:6A4F59AA |
| html-safe-nonce | 8c2a1bf503b896519a11ae9dc43fd90cf458e2381e033caade81363dab43f3a8 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4M0Q2OjExNTk3RDoxMDA5QUZCOjE2QjUwRDg6NkE0RjU5QUEiLCJ2aXNpdG9yX2lkIjoiODEyOTIxODkxNTE0MDA2NTcwNiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 638756b9a8bea0b5551e80b9e64e1ab97e4f3e8cec5e01f019b535fbf628a5f9 |
| hovercard-subject-tag | pull_request:2449144217 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/inducer/arraycontext/pull/313/files |
| twitter:image | https://avatars.githubusercontent.com/u/7232918?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/7232918?s=400&v=4 |
| og:image:alt | In some mirgecom examples, dt can be a DOFArray. When using _compiled_lsrk45_step from grudge (which now uses BcastUntilActxArray for h, etc.), this causes dt to broadcast down to the array context... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | b92d11c0aa4a77d54ef4af1078b6a15fb5a70a215b30c4ecf28889d5a8e656d9 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/inducer/arraycontext git https://github.com/inducer/arraycontext.git |
| octolytics-dimension-user_id | 352067 |
| octolytics-dimension-user_login | inducer |
| octolytics-dimension-repository_id | 364716587 |
| octolytics-dimension-repository_nwo | inducer/arraycontext |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 364716587 |
| octolytics-dimension-repository_network_root_nwo | inducer/arraycontext |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 4b249b445842943ed31549e027f57a8ade9881ed |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width