search

keklikyusuf / DeltaElektronika

Delta Elektronika Power Supply Controller
MIT License
3 stars 1 forks source link

To high a voltage :( #3

Closed BobbysGithub closed 3 years ago

BobbysGithub commented 3 years ago

Wow bug's already fixed, don't you need sleep?

Just fire'd up the charger and noted that the battery voltage spike'd above the set value. It corrected itself at the next sample but it seems kinda strange since the set voltage is 3.65V

Good ting i had the debug running ;) And even better there is a debug log! 3.7V @ startup.log

image

keklikyusuf commented 3 years ago

Hi Bobby, No I do not sleep at all :)

It takes instant measurements from delta, there is no averaging it looks like when system started, till it is able to regulate the output it had a spike moment. There voltage hard limits has been implemented with margin of 0.5V upper than set point. For voltage only, I would like to set it same with your set point to prevent spike moments.

I think for battery we do not want a margin on the voltage that we can keep it as same with set point to keep operation safe.

With v0.0.6, voltage marging has been changed from python SystemSubsystem(self.IPV4).SetVoltageLimit(self.bulkVoltage + 0.5, 'ON') to python SystemSubsystem(self.IPV4).SetVoltageLimit(self.bulkVoltage, 'ON').

I hope it can solve spike moments at regulation moments of the device :)

Cheers 💯 @keklikyusuf