search

kanflo / opendps

Give your DPS5005 the upgrade it deserves
MIT License
895 stars 124 forks source link

Make func_cv optional #170

Closed kohrar closed 3 years ago

kohrar commented 5 years ago

The func_cv screen was not optional despite the fact that func_cl is the combined version of func_cc and func_cv. The Makefile now supports the CV_ENABLE flag to toggle this screen.

Because cl combines features in cc and cv, both cc and cv are now disabled by default in favor for the cl screen.

Xenoamor commented 5 years ago

Much appreciated. This resolves #133

kanflo commented 5 years ago

At this point I would actually like to keep CV by default but I see a future where it's optional. My rationale is the same as mentioned here:

The only reason for keeping CV mode, is if you like the PSU to shutdown when reaching the limit instead of limiting the amperage. I can se a usecase for that when you are working on equipment where undervoltage could damage the device.

That would be my use case, powering micro controllers. If the amp limit is hit, power needs to be cut or my uC gets toasted ;)

When the future OpenDPS wifi gets written (as there is too much information both accurate and obsolete in my old blog posts) there could be a section explaining the different modes and their use cases. As I understand, the CL mode is very useful for charging batteries (please correct me if I am wrong).

frederikvs commented 3 years ago

I've encountered a small issue with this branch : when you configure it so that FUNCGEN is the only screen that's enabled, and then you switch to "another" screen, you end up with a screen that's mostly black. Only the sine wave and the input voltage are shown, all else has been removed.

Other than that, I think it's a good idea to have func_CV be optional :-)