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 2024-11-01 #2863

Merged
merged 12 commits into from
Nov 1, 2024
Merged

FEMS Backports 2024-11-01 #2863

merged 12 commits into from
Nov 1, 2024

Conversation

sfeilmeier
Copy link
Contributor

@sfeilmeier sfeilmeier commented Nov 1, 2024

  • UI: Label correction in SoC widget in history.
    • Fixed lables shown below.
    • Energy is represented as Power.
    • Entladung/Beladung is represented as Charge/Discharge power. replaced with Just Charge/Discharge.
  • Keba ReadWorker: fix possible Exception
    • Added Tests to identify when and how the exception can occur.
    • Tests for the "calculateCycleTime" work for multiple cases without throwing an exception
    • Tests could still be kept for code coverage
  • GoodWe: improve serial number filter
    • Change the serial number filter according to the goowe serial number rule.
    • Each GoodWe Type is now detected individually by its rated power in combination with its internal model series code.
  • GoodWeGridMeter: adjust CURRENT sign current if needed
    • Added Listener to ActivePowerLX Channels
    • Each Listeners checks if the nextValue for the power channel if the current and power have the same sign
    • If the sign is different the current channel is set with * -1
  • JUnit Test Framework: add possibility to test raw modbus reads
    • DummyModbusBridge can now be used to test raw modbus reads
    • Apply new test for changes in GoodWe GridMeter and SunSpec
    • Also: Improve and apply ReflectionUtils
  • Replace exchangerate.host with Eurpean Central Bank API
  • _host/OsVersion: read os version as string
    • Added OS_VERSION Channel in Host Component
    • Write e. g. "11 (bullseye)" in channel for linux and for windows "Windows 11"
  • UI: replace currentUser observable with signal and solve login caching
    • Start recommended restructuring observables to signals
    • Fix UserComponent caching user information after logout
  • UI: Smaller Improvements
    • Auto capitalize ibn-installer key input field to show only upper case letters in mobile view (keyboard)
    • Fix wrong service number in offline manuals
    • adjust time of use chart in live with replacing the upper most left yAxis tick with the chart title
  • UI: remove app center app icon
    • Improving app center readability of icons
  • Time-of-use-Tariff Swisspower

Co-authored-by: Lukas Rieger [email protected]
Co-authored-by: Sagar Venu [email protected]
Co-authored-by: Sebastian Asen [email protected]
Co-authored-by: Stefan Feilmeier [email protected]
Co-authored-by: Johann Kaufmann [email protected]
Co-authored-by: Michael Grill [email protected]

sfeilmeier and others added 12 commits November 1, 2024 16:15
Fixed lables shown below.

-> Energy is represented as Power.
-> Entladung/Beladung is represented as Charge/Discharge power. replaced with Just Charge/Discharge.

Co-authored-by: Lukas Rieger <[email protected]>
Reviewed-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 Tests to identify when and how the exception can occur.
- Tests for the "calculateCycleTime" work for multiple cases without throwing an exception
- Tests could still be kept for code coverage

Co-authored-by: Sebastian Asen <[email protected]>
Co-authored-by: Stefan Feilmeier <[email protected]>
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]>
- Change the serial number filter according to the goowe serial number rule.
- Each GoodWe Type is now detected individually by its rated power in combination with its internal model series code.

Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Sebastian Asen <[email protected]>
Co-committed-by: Sebastian Asen <[email protected]>
- Added Listener to ActivePowerLX Channels
- Each Listeners checks if the nextValue for the power channel if the current and power have the same sign
- If the sign is different the current channel is set with * -1

Co-authored-by: Stefan Feilmeier <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Johann Kaufmann <[email protected]>
Co-committed-by: Johann Kaufmann <[email protected]>
- `DummyModbusBridge` can now be used to test raw modbus reads
- Apply new test for changes in GoodWe GridMeter and SunSpec

Also:
- Improve and apply `ReflectionUtils`

Reviewed-by: Sebastian Asen <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Sagar Venu <[email protected]>
Co-committed-by: Sagar Venu <[email protected]>
- Added OS_VERSION Channel in Host Component
- Write e. g. "11 (bullseye)" in channel for linux and for windows "Windows 11"

Reviewed-by: Kai Jeschek <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Michael Grill <[email protected]>
Co-committed-by: Michael Grill <[email protected]>
- Start recommended restructuring observables to signals
- Fix `UserComponent` caching user information after logout

Reviewed-by: Sagar Venu <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Lukas Rieger <[email protected]>
Co-committed-by: Lukas Rieger <[email protected]>
- Auto capitalize ibn-installer key input field to show only upper case letters in mobile view (keyboard)
- Fix wrong service number in offline manuals
- adjust time of use chart in live with replacing the upper most left yAxis tick with the chart title

Reviewed-by: Hanna Späth <[email protected]>
Reviewed-by: Sagar Venu <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Lukas Rieger <[email protected]>
Co-committed-by: Lukas Rieger <[email protected]>
- Improving app center readability of icons

Reviewed-by: Michael Grill <[email protected]>
Co-authored-by: Lukas Rieger <[email protected]>
Co-committed-by: Lukas Rieger <[email protected]>
Co-authored-by: Michael Grill <[email protected]>
Reviewed-by: Michael Grill <[email protected]>
Reviewed-by: Stefan Feilmeier <[email protected]>
Co-authored-by: Sagar Venu <[email protected]>
Co-committed-by: Sagar Venu <[email protected]>
Copy link

codecov bot commented Nov 1, 2024

Codecov Report

Attention: Patch coverage is 49.66741% with 227 lines in your changes missing coverage. Please review.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #2863      +/-   ##
=============================================
+ Coverage      56.66%   56.84%   +0.18%     
- Complexity      8926     9050     +124     
=============================================
  Files           2167     2171       +4     
  Lines          91959    92271     +312     
  Branches        6831     6848      +17     
=============================================
+ Hits           52103    52441     +338     
+ Misses         37981    37910      -71     
- Partials        1875     1920      +45     

@sfeilmeier sfeilmeier merged commit ae1dbf8 into develop Nov 1, 2024
6 of 7 checks passed
@sfeilmeier sfeilmeier deleted the feature/fems-backports branch November 1, 2024 16:51
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