aaronsb / REC-BMSManager

Powershell based management for REC-BMS battery management
GNU Lesser General Public License v3.0
6 stars 0 forks source link

Get Parameter works but Set Parameter not #4

Closed gnagflow80 closed 3 years ago

gnagflow80 commented 3 years ago

Hi Aaron, 1st - it is realy great what you produce and share!!! I experienced, that it is required to add the Group Serialout, if you use powershell for linux that it works. But, only the get-BMSParameter works. When i use Set-BMSParameter it looks like: set-bms

It doesnt take the value, no matter iv you give the right confirmation instruction or not.

gnagflow80 commented 3 years ago

It works only not on "CAPA", but it works on CLOW. Obviously there are some parameters changeable by your sofware, some not.

aaronsb commented 3 years ago

Can you try again and set the capa value to 1070.0 instead of 1070? Or otherwise cast it to float? I remained very strict about number types so you don't accidentally cast one to another and cause misadventure.

Perhaps an improvement would be to indicate that there was a number mismatch (int to float for example)

image

gnagflow80 commented 3 years ago

Can you try again and set the capa value to 1070.0 instead of 1070? Or otherwise cast it to float? I remained very strict about number types so you don't accidentally cast one to another and cause misadventure.

Perhaps an improvement would be to indicate that there was a number mismatch (int to float for example)

image

Oh I see its a float. Alright, it is working now! Thanks a lot. --> now i try the MQTT & Grafana thing.. grafik