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

Invert siemens pac1600 #2821

Closed
wants to merge 5 commits into from

Conversation

JosefRick
Copy link
Contributor

Implement the invert task of Siemens PAC 1600 and corrected the production and consumption energy register. These were swapped. It also implement the SCALE_FACTOR_MINUS_2_AND_INVERT, because it hasn't exist.

Copy link

codecov bot commented Sep 27, 2024

Codecov Report

Attention: Patch coverage is 60.78431% with 20 lines in your changes missing coverage. Please review.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #2821      +/-   ##
=============================================
- Coverage      56.01%   56.01%   -0.00%     
- Complexity      8197     8200       +3     
=============================================
  Files           2103     2103              
  Lines          89017    89050      +33     
  Branches        6560     6561       +1     
=============================================
+ Hits           49853    49870      +17     
- Misses         37446    37460      +14     
- Partials        1718     1720       +2     

Changes the position of a bracket because it caused an error in the build check.
@JosefRick JosefRick closed this Sep 27, 2024
@JosefRick JosefRick deleted the InvertSiemensPAC1600 branch September 27, 2024 12:16
@JosefRick JosefRick restored the InvertSiemensPAC1600 branch September 30, 2024 15:15
@JosefRick JosefRick reopened this Sep 30, 2024
@JosefRick JosefRick closed this Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant