Skip to content

Fix parametric number of runs updates for hybrids #1541#1639

Merged
sjanzou merged 1 commit into
developfrom
SAM_1541
Dec 6, 2023
Merged

Fix parametric number of runs updates for hybrids #1541#1639
sjanzou merged 1 commit into
developfrom
SAM_1541

Conversation

@sjanzou

@sjanzou sjanzou commented Dec 6, 2023

Copy link
Copy Markdown
Collaborator

Test file with hybrids and non-hybrids
sam_1541.zip

@sjanzou sjanzou self-assigned this Dec 6, 2023
@sjanzou sjanzou linked an issue Dec 6, 2023 that may be closed by this pull request
@cpaulgilman cpaulgilman added the dev bug Internal bug that should not be included in release notes label Dec 6, 2023
@cpaulgilman cpaulgilman added this to the SAM Fall 2023 Release milestone Dec 6, 2023

@cpaulgilman cpaulgilman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tested adding rows to parametric rows and running parametric simulations for hybrid configurations.

Note that total number of simulations (n of total) shown in "Preparing Simulations" window is incorrect when running simulations immediately after changing number of rows.

@brtietz brtietz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Works for me as well, thank you!

@sjanzou sjanzou merged commit 9b7a284 into develop Dec 6, 2023
@sjanzou sjanzou deleted the SAM_1541 branch December 6, 2023 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev bug Internal bug that should not be included in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hybrids Parametrics Freezes when adjusting number of input rows

3 participants