search

labgrid-project / labgrid

Embedded systems control library for development, testing and installation
https://labgrid.readthedocs.io/
Other
343 stars 177 forks source link

driver/power: Add support for Gude 87-1210-18 #1534

Closed SmithChart closed 3 weeks ago

SmithChart commented 3 weeks ago

Description

The Gude Expert Power Control 87-1210-18 is a vertical 20 port power distribution unit: https://gude-systems.com/en/products/expert-power-control-87-1210/

It uses the same HTTP-API, as the Gude Power Control 8031 - but simply has more ports. This PR adds support and documentation for this device.

While I am on it: Re-format the file with ruff and replace the Windows Line-Endings.

Change has been tested locally against a Gude 87-1210-18.

Checklist

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.

Project coverage is 56.6%. Comparing base (71bfcd2) to head (af5a3bf). Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
labgrid/driver/power/gude8031.py 0.0% 15 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1534 +/- ## ======================================== - Coverage 56.6% 56.6% -0.1% ======================================== Files 168 168 Lines 13033 13033 ======================================== - Hits 7385 7384 -1 - Misses 5648 5649 +1 ``` | [Flag](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1534/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | Coverage Δ | | |---|---|---| | [3.10](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1534/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.6% <0.0%> (ø)` | | | [3.11](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1534/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.6% <0.0%> (ø)` | | | [3.12](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1534/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.6% <0.0%> (ø)` | | | [3.13](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1534/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `56.6% <0.0%> (ø)` | | | [3.9](https://app.codecov.io/gh/labgrid-project/labgrid/pull/1534/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=labgrid-project#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

SmithChart commented 3 weeks ago

Updated the commit messages as suggested.