Skip to content

Fixed clipped system power battery charging option for Generic Battery systems#677

Merged
mjprilliman merged 1 commit into
developfrom
sam-646-fix
Aug 9, 2021
Merged

Fixed clipped system power battery charging option for Generic Battery systems#677
mjprilliman merged 1 commit into
developfrom
sam-646-fix

Conversation

@mjprilliman

Copy link
Copy Markdown
Collaborator

-Changed State default from 1 to 0 as no clipped power goes to battery for AC connected Generic Battery systems
-Added State check in callbacks
-Closes #646

@mjprilliman mjprilliman added this to the SAM Fall 2021 Release milestone Aug 8, 2021
@mjprilliman mjprilliman requested a review from brtietz August 8, 2021 19:17
@mjprilliman mjprilliman self-assigned this Aug 8, 2021

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

Looks good! Thanks for doing this fix!

@mjprilliman mjprilliman merged commit 212e4b6 into develop Aug 9, 2021
@mjprilliman mjprilliman deleted the sam-646-fix branch August 9, 2021 18:20
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

added to release notes PR and/or issue has been added to release notes for a public release battery

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generic system shows "battery can charged from clipped power" which doesn't make sense for AC connected systems

3 participants