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.

Sign up for GitHub

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

Jump to bottom

Feature/reverse power relay #2684

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

Feature/reverse power relay #2684

wants to merge 18 commits into from

Conversation

DerWahreKlinki
Copy link
Collaborator

@DerWahreKlinki DerWahreKlinki commented Jun 26, 2024 •
edited
Loading

Controller to react to a reverse power relay (german: Rundsteuerempfänger)

Tested with Siemens LOGO! 8 as network -> modbus interface

@OpenEMS OpenEMS deleted a comment from codecov bot Jun 26, 2024
@codecov Codecov
Copy link

codecov bot commented Jun 27, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@DerWahreKlinki
Copy link
Collaborator Author

  • Made max power of PV inverter configurable (not all inverters have set this value) and the 30/60% are calculated dynamically
  • Improved handling of empty values
  • Improved JuniTests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sn0w3y Sn0w3y Awaiting requested review from Sn0w3y

At least 1 approving review is required to merge this pull request.

Assignees

@sfeilmeier sfeilmeier

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DerWahreKlinki @Sn0w3y @sfeilmeier