Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A common problem with banks is that members of the bank may need to each have its own set of parameter values. The only mechanism we currently offer requires modifying the reactor class itself, which needs to set state variables (rather than parameters) from a global table. This means that the reactor can only be used in a bank and must be written to work in bank.
This PR adds to the C and Python targets the ability to initialize parameters using a table lookup and the
bank_index
variable. Here is an example:The
table
defined in the preamble determines the parameter values to use for each bank member.This change was motivated by @Soroosh129's PacMan game example, where I have made the ghosts into a bank rather than four individual reactor instances.