Skip to content

[Edge] Add support for SDM72D-M-2 meter#3491

Merged
sfeilmeier merged 2 commits intoOpenEMS:developfrom
Sn0w3y:sdm-630-compatibility
Dec 25, 2025
Merged

[Edge] Add support for SDM72D-M-2 meter#3491
sfeilmeier merged 2 commits intoOpenEMS:developfrom
Sn0w3y:sdm-630-compatibility

Conversation

@Sn0w3y
Copy link
Copy Markdown
Collaborator

@Sn0w3y Sn0w3y commented Dec 24, 2025

This commit updates the Eastron SDM630 meter implementation to include support for the SDM72D-M-2 meter. Changes were made in both the readme documentation to list the SDM72D-M-2 as a compatible device and in the Config.java to reflect this compatibility in the description. This enhancement allows users of the SDM72D-M-2 meter to utilize the existing SDM630 implementation, broadening the applicability of the driver.

This commit updates the Eastron SDM630 meter implementation to include support for the SDM72D-M-2 meter. Changes were made in both the readme documentation to list the SDM72D-M-2 as a compatible device and in the Config.java to reflect this compatibility in the description. This enhancement allows users of the SDM72D-M-2 meter to utilize the existing SDM630 implementation, broadening the applicability of the driver.
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #3491      +/-   ##
=============================================
- Coverage      59.75%   59.74%   -0.00%     
  Complexity       105      105              
=============================================
  Files           2952     2952              
  Lines         127747   127747              
  Branches        9551     9551              
=============================================
- Hits           76325    76315      -10     
- Misses         48521    48524       +3     
- Partials        2901     2908       +7     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sfeilmeier
Copy link
Copy Markdown
Contributor

It looks like the protocol could be the same for all three-phase meters?! If that's true, we could rename the packages and Config descriptions accordingly - see SOCOMEC implementation.

For now this is ok here and I'll merge. Thanks! 👍

@sfeilmeier sfeilmeier merged commit caecc4b into OpenEMS:develop Dec 25, 2025
7 checks passed
@Sn0w3y Sn0w3y deleted the sdm-630-compatibility branch December 25, 2025 15:10
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.

2 participants