Skip to content

Addtl lifetime testing in integration tests#677

Merged
sjanzou merged 2 commits into
developfrom
addtl_lifetime_testing_in_integration_tests
Nov 30, 2021
Merged

Addtl lifetime testing in integration tests#677
sjanzou merged 2 commits into
developfrom
addtl_lifetime_testing_in_integration_tests

Conversation

@brtietz

@brtietz brtietz commented Nov 29, 2021

Copy link
Copy Markdown
Collaborator

Add additional tests of battery lifetime to pin down issue #675 on price signals dispatch. Increase the tolerance for the price signals test until we can better nail down cross platform differences.

Will re-open #614 to solve after the release.

@sjanzou sjanzou 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.

Looks good

@sjanzou sjanzou merged commit a90a998 into develop Nov 30, 2021
@sjanzou sjanzou deleted the addtl_lifetime_testing_in_integration_tests branch November 30, 2021 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CMPvsamv1BatteryIntegration_cmod_pvsamv1.ResidentialDCBatteryModelPriceSignalDispatch faling on Travis and runs fine on Windows

3 participants