Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[Hotfix] fix restore after failing batch scenario #462

Merged
merged 1 commit into from Jan 9, 2024

Conversation

mgovers
Copy link
Member

@mgovers mgovers commented Jan 9, 2024

Fixes issue: #461

Changes proposed in this PR include:

This is a hotfix that does not care about performance.

Approprate sustainable solution will be done in #460 .

Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers added the bug Something isn't working label Jan 9, 2024
@mgovers mgovers self-assigned this Jan 9, 2024
@mgovers
Copy link
Member Author

mgovers commented Jan 9, 2024

Failing CI is due to boost artifactory bug boostorg/boost#842

Copy link
Member

@TonyXiang8787 TonyXiang8787 left a comment

Choose a reason for hiding this comment

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

Approved.

Just to be sure on the reasoning. Because we do not need to worry about performance for error scenario, so we choose to copy the model (less efficient), other than try to restore the components (more complicated).

@mgovers
Copy link
Member Author

mgovers commented Jan 9, 2024

Approved.

Just to be sure on the reasoning. Because we do not need to worry about performance for error scenario, so we choose to copy the model (less efficient), other than try to restore the components (more complicated).

yes indeed. In the near future, a sustainable solution will improve on this. Since this is a hotfix, fixing functionality is more important than performance

@mgovers
Copy link
Member Author

mgovers commented Jan 9, 2024

Failing CI is due to boost artifactory bug boostorg/boost#842

re-running after TonyXiang8787/pybuild-header-dependency#3 fixed the boost URL

Copy link

sonarcloud bot commented Jan 9, 2024

@mgovers mgovers linked an issue Jan 9, 2024 that may be closed by this pull request
@mgovers mgovers merged commit f0bcfd5 into main Jan 9, 2024
27 checks passed
@mgovers mgovers deleted the bugfix/add-restore-fallback branch January 9, 2024 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] non-independent scenarios when batch scenario fails
2 participants