Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make "RCV Time" and "RFV Time" controllable #548

Closed
dmode9000 opened this issue Jul 27, 2024 · 8 comments · Fixed by #549
Closed

Make "RCV Time" and "RFV Time" controllable #548

dmode9000 opened this issue Jul 27, 2024 · 8 comments · Fixed by #549
Labels
enhancement New feature or request v14

Comments

@dmode9000
Copy link

Could you please let me know if the "RCV Time" and "RFV Time" parameters are available? I haven't found them in the HA interface yet and haven't found any information on how to add them to the interface.

@syssi
Copy link
Owner

syssi commented Jul 29, 2024

How do you talk to your BMS? If you are using BLE the feature was added here: #422

You can just add these additional number entities to your configuration YAML: https://github.com/syssi/esphome-jk-bms/blob/main/esp32-ble-v14-example.yaml#L132-L135

@syssi
Copy link
Owner

syssi commented Jul 29, 2024

Oh. We are talking about the runtimes. This setting isn't implemented yet.

# RCV time
aa:55:90:eb: b3 :01: 14 :71:55:0b:ba:97:e6:68:67:fe:71:5b:50:33 # 20 * 0.1f = 2.0h
aa:55:90:eb: b3 :01: 0a :76:55:c2:22:63:4f:02:ce:a8:d8:a7:0a:9a # 10 * 0.1f = 1.0h

# RFV time
aa:55:90:eb: b4 :01: 5a :45:6a:43:1a:4d:85:5f:3e:db:6a:f8:73:b4 # 90 * 0.1f = 9.0h
aa:55:90:eb: b4 :01: 50 :d1:61:5a:d0:53:35:21:c9:8a:63:ec:6d:93 # 80 * 0.1f = 8.0h

This issue is the follow up of #422 because two feature was missing.

@syssi syssi added the enhancement New feature or request label Jul 29, 2024
@syssi syssi changed the title JK-PBx - Where are the "RCV Time" and "RFV Time" params/values? Make "RCV Time" and "RFV Time" controllable Jul 29, 2024
@syssi syssi added the v14 label Jul 29, 2024
@syssi syssi linked a pull request Jul 29, 2024 that will close this issue
@syssi
Copy link
Owner

syssi commented Jul 29, 2024

I've started to implement the feature but some testing is required to complete the feature. Are you able to talk to your BMS already? Could you offer some testing?

@dmode9000
Copy link
Author

dmode9000 commented Jul 30, 2024

Yes, I can test it. Do I understand correctly that after recompiling and flashing in Homeassisant, these parameters should appear as available?

After recompiling, the RCV and RFV parameters (NOT time but Voltage) started displaying in HA as 1 and 0.4 volts. While in fact, they did not change and remained above 3V.
I also did not find the "RCV Time" and "RFV Time" parameters in HA.

@syssi
Copy link
Owner

syssi commented Jul 30, 2024

You have to extend your YAML configuration be these 4 lines at the number section:

https://github.com/syssi/esphome-jk-bms/blob/main/esp32-ble-v14-example.yaml#L136-L139

number:
  - platform: jk_bms_ble
    jk_bms_ble_id: bms0
    # ...
    cell_request_charge_voltage_time:
      name: "${name} cell request charge voltage time"
    cell_request_float_voltage_time:
      name: "${name} cell request float voltage time"

And recompile + flash your ESP. The two number entities are populated as soon a device info frame gets received. This frame is retrieved once on an established BLE connection and can be trieved manually using the retrieve device info button.

@dmode9000
Copy link
Author

dmode9000 commented Jul 30, 2024

Recompiled and added 4 lines in the number section.
"RCV Time" and "RFV Time" parameters are visible in the interface but are not retrieved from the BMS, even after manually clicking on "Retrieve Device Info."

New issues::

  • RCV and RFV parameters (NOT time but Voltage) started displaying fake values in HA as 1 and 0.4 volts.

image

  • The heating option on BMS is constantly turning off and on.

image

@syssi
Copy link
Owner

syssi commented Jul 31, 2024

My fault. I've pushed a fix for the RCV, RCV time, RFV and RFV time entities. Please recompile & flash again. Could you create a new issue for the heating sensor?

@dmode9000
Copy link
Author

Recompiled, RCV, RCV time, RFV and RFV time entities now Ok.

New heating sensor issue created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v14
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants