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

FEMS Backports 2025-01-16 #2969

Merged
merged 15 commits into from
Jan 16, 2025
Merged

FEMS Backports 2025-01-16 #2969

merged 15 commits into from
Jan 16, 2025

Conversation

sfeilmeier
Copy link
Contributor

@sfeilmeier sfeilmeier commented Jan 16, 2025

  • Edge

    • JsonUtils: Update switch cases with pattern matching
    • HttpBridge: add detailed Logger
      • Moved DebugMode to Common
      • Added DebugMode to BridgeHttp
      • Added parameter of type DebugMode to NetworkEndpointFetcher
      • Added Detailed log of result in fetchEndpoint if DebugMode is DETAILED
      • Set DebugMode to DETAILED in HardyBarthEvcsImpl if debug is true in config
    • Sum + ESS: update Modbus table descriptions
      • Adds and changes descriptions of the modbus tables
    • KEBA: fix issue with SessionEnergy
      • Add KEBA specific R2State-enum (for "Report 2")
      • KEBA: drop CHARGING_REJECTED
      • SessionEnergy is set to 0 on UNPLUGGED event
    • Modbus-Api-Controller: add RTU and unify controllers to abstract superclass
      • Implementation of ModbusRTU
      • Refactoring of ModbusTCP to avoid redundant classes
      • Implementation of ModbusRTU App
      • Added "Download Protocol" button for ModbusRtu
    • AppCenter Home: NA-Protection changed text
      • changed na-protection text
    • IO Weidmüller: Added UR20_16DI_P module
    • Symmetric Battery Inverter: add cabinet temperature channel
      • Added a "TEMPERATURE_CABINET" channel in the Battery inverter interface.
    • HardyBarth: temporary fix for null responses
      • This fixes an issue with interrupted charging sessions due to null values.
      • added check for null values in HardyBarthReadUtils
      • The first 3 null values are ignored and no values are written in these cycles
      • Afterwards null values are written in channels as usual
    • AppCenter: add estimated configuration endpoint for IBN
      • Added endpoint to retrieve estimated installation configuration
    • AppCenter: Meta App + Update Component via apps
      • Added core app for meta component
      • Added Logic for updating component config via their apps
    • EmergencyReserveFromGrid: start charging without delay from ramp
      • Added check with previous state that ignores ramp for 1 cycle
  • UI

    • Prepare Login Improvements
    • Added template to override the default formly behaviour bugs
    • fix styling of negative-values and positive-values info

sfeilmeier and others added 15 commits January 16, 2025 21:29
Reviewed-by: Michael Grill <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Kai Jeschek <[email protected]>
Co-committed-by: Kai Jeschek <[email protected]>
Reviewed-by: Lukas Rieger <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: FENECON\anas.shetla <[email protected]>
Co-committed-by: FENECON\anas.shetla <[email protected]>
- Moved DebugMode to Common
- Added DebugMode to BridgeHttp
- Added parameter of type DebugMode to NetworkEndpointFetcher
- Added Detailed log of result in fetchEndpoint if DebugMode is DETAILED
- Set DebugMode to DETAILED in HardyBarthEvcsImpl if debug is true in config

Reviewed-by: Michael Grill <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Johann Kaufmann <[email protected]>
Co-committed-by: Johann Kaufmann <[email protected]>
Adds and changes descriptions of the modbus tables

Reviewed-by: Sebastian Asen <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Fabian Brandtner <[email protected]>
Co-committed-by: Fabian Brandtner <[email protected]>
- Add KEBA specific R2State-enum (for "Report 2")
- KEBA: drop CHARGING_REJECTED
- SessionEnergy is set to 0 on UNPLUGGED event

Co-authored-by: Johann Kaufmann <[email protected]>
Reviewed-by: Sebastian Asen <[email protected]>
Reviewed-by: Johann Kaufmann <[email protected]>
…rclass

- Implementation of ModbusRTU
- Refactoring of ModbusTCP to avoid redundant classes
- Implementation of ModbusRTU App
- Added "Download Protocol" button for ModbusRtu

Co-authored-by: Sebastian Asen <[email protected]>
Co-authored-by: Stefan Feilmeier <[email protected]>
Reviewed-by: Lukas Rieger <[email protected]>
Reviewed-by: Michael Grill <[email protected]>
Co-authored-by: Fabian Brandtner <[email protected]>
Co-committed-by: Fabian Brandtner <[email protected]>
- changed na-protection text

Reviewed-by: Johann Kaufmann <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Michael Grill <[email protected]>
Co-committed-by: Michael Grill <[email protected]>
Co-authored-by: Pooran Chandrashekaraiah <[email protected]>
Reviewed-by: Hueseyin Sahutoglu <[email protected]>
Reviewed-by: Michael Grill <[email protected]>
Co-authored-by: Lukas Rieger <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Sagar Venu <[email protected]>
Co-committed-by: Sagar Venu <[email protected]>
Added a "TEMPERATURE_CABINET" channel in the Battery inverter interface.

Reviewed-by: Michael Grill <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Pooran Chandrashekaraiah <[email protected]>
Co-committed-by: Pooran Chandrashekaraiah <[email protected]>
This fixes an issue with interrupted charging sessions due to `null` values.

- added check for null values in HardyBarthReadUtils
- The first 3 null values are ignored and no values are written in these cycles
- Afterwards null values are written in channels as usual

Reviewed-by: Sebastian Asen <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Johann Kaufmann <[email protected]>
Co-committed-by: Johann Kaufmann <[email protected]>
Added endpoint to retrieve estimated installation configuration

Reviewed-by: Johann Kaufmann <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Michael Grill <[email protected]>
Co-committed-by: Michael Grill <[email protected]>
- fix styling of `negative-values` and `positive-values` info
- restore old visibility of energy journey (only being visible for `home-10`)

Reviewed-by: Sebastian Asen <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Lukas Rieger <[email protected]>
Co-committed-by: Lukas Rieger <[email protected]>
- Added core app for meta component
- Added Logic for updating component config via their apps

Co-authored-by: Michael Grill <[email protected]>
Reviewed-by: Michael Grill <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Johann Kaufmann <[email protected]>
Co-committed-by: Johann Kaufmann <[email protected]>
Added check with previous state that ignores ramp for 1 cycle

Reviewed-by: Sebastian Asen <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Johann Kaufmann <[email protected]>
Co-committed-by: Johann Kaufmann <[email protected]>
@sfeilmeier sfeilmeier merged commit 79650cb into develop Jan 16, 2025
2 checks passed
@sfeilmeier sfeilmeier deleted the feature/fems-backports branch January 16, 2025 20:54
Copy link

codecov bot commented Jan 16, 2025

Codecov Report

Attention: Patch coverage is 34.95466% with 789 lines in your changes missing coverage. Please review.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #2969      +/-   ##
=============================================
- Coverage      57.08%   56.86%   -0.22%     
- Complexity      9732     9804      +72     
=============================================
  Files           2266     2281      +15     
  Lines          96819    97624     +805     
  Branches        7163     7208      +45     
=============================================
+ Hits           55262    55501     +239     
- Misses         39494    40038     +544     
- Partials        2063     2085      +22     

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