Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed an issue with bank_index #1087

Merged
merged 1 commit into from
Apr 11, 2022
Merged

Fixed an issue with bank_index #1087

merged 1 commit into from
Apr 11, 2022

Conversation

Soroosh129
Copy link
Contributor

Fixed an issue where bank_index was not being generated as a reactor parameter unless the user had explicitly added it as a parameter.

Where bank_index was not being generated unless the user explicitly added it as a reactor property
@Soroosh129 Soroosh129 requested a review from housengw April 10, 2022 05:10
Copy link
Contributor

@housengw housengw left a comment

Choose a reason for hiding this comment

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

LGTM

@housengw housengw merged commit 794fabc into master Apr 11, 2022
@lhstrh lhstrh added the bug Something isn't working label Apr 11, 2022
@lhstrh lhstrh deleted the python-bank-index-fix branch April 11, 2022 17:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants