You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
plate doesn't take an event_dim argument, so I thought your comment was referring to the dim argument of plate. Did you mean the event_dim argument of param, or something else?
Yes - that's what I meant. I can start looking at implementing the event_dim parameter of param if you think that'd be useful
rvs314
changed the title
Support nested plates with manual dim in minipyro example
Support param arguments with event_dim in minipyro example
Jul 2, 2024
rvs314
changed the title
Support param arguments with event_dim in minipyro example
Support param calls with event_dim in minipyro example
Jul 2, 2024
Ah, OK. I don't think that's as interesting in terms of exercising effectful functionality, so let's not bother for now. I'll go ahead and close this issue.
There is at least one test in
pyroapi
that uses nested plates with manualdim
arguments.This exercises some nontrivial effect handler behavior, so after #8 lands, we should update our minipyro implementation to make those tests pass.
The text was updated successfully, but these errors were encountered: