Skip to content

Add Kiprim DC310S power supply driver#471

Open
kerby2000 wants to merge 4 commits intoinstrumentkit:mainfrom
kerby2000:add-kiprim-dc310s
Open

Add Kiprim DC310S power supply driver#471
kerby2000 wants to merge 4 commits intoinstrumentkit:mainfrom
kerby2000:add-kiprim-dc310s

Conversation

@kerby2000
Copy link

Add support for the Kiprim DC310S single-output serial power supply.

This adds a conservative InstrumentKit-style driver with:

  • voltage and current setpoints
  • output enable/disable
  • measured voltage, current, and power readback
  • documented 0-30 V and 0-10 A bounds on setpoints

Also add transcript-based unit tests, package exports, API reference docs, and a minimal example script.

kerby2000 and others added 2 commits March 8, 2026 18:01
Add support for the Kiprim DC310S single-output serial power supply.

This adds a conservative InstrumentKit-style driver with:
- voltage and current setpoints
- output enable/disable
- measured voltage, current, and power readback
- documented 0-30 V and 0-10 A bounds on setpoints

Also add transcript-based unit tests, package exports, API reference docs,
and a minimal example script.
@scasagrande
Copy link
Contributor

seems reasonable

I'm aware I don't a policy posted yet, but was any of this LLM generated?

@codecov
Copy link

codecov bot commented Mar 8, 2026

Codecov Report

❌ Patch coverage is 97.36842% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 99.19%. Comparing base (34f82c7) to head (07c1ccb).

Files with missing lines Patch % Lines
src/instruments/kiprim/dc310s.py 97.36% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #471      +/-   ##
==========================================
- Coverage   99.20%   99.19%   -0.01%     
==========================================
  Files          96       97       +1     
  Lines        9648     9686      +38     
==========================================
+ Hits         9571     9608      +37     
- Misses         77       78       +1     
Flag Coverage Δ
unittests 99.19% <97.36%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kerby2000
Copy link
Author

seems reasonable

I'm aware I don't a policy posted yet, but was any of this LLM generated?

Nowadays most of the code is LLM generated. Under good guidance :)
Should I add an explicit remark about LLL generated code?

@scasagrande
Copy link
Contributor

You always should. But I'm also going to have to think about its inclusion, sorry

@kerby2000
Copy link
Author

I'm using both APIs this and OWON SDS1104 oscilloscope for my projects so if something will pop up I will commit the updates.
It took me quite a while to get some OWON SDS1104 SCPI commands working as they are not that well documented.

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