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

Readout of Victron DC/DC charger Orion XS #50

Closed
terdt opened this issue May 22, 2024 · 15 comments · Fixed by #54
Closed

Readout of Victron DC/DC charger Orion XS #50

terdt opened this issue May 22, 2024 · 15 comments · Fixed by #54

Comments

@terdt
Copy link

terdt commented May 22, 2024

I

@terdt terdt changed the title Readout of Victron DC/DC Orion XS Readout of Victron DC/DC charger Orion XS May 22, 2024
@terdt
Copy link
Author

terdt commented May 22, 2024

I replaced the Orion-Tr Smart DC/DC by the new Orion XS charger. In the yaml I replaced the mac address and the encryption key by the Orion XS data. But it's not working out of the box as I expected. What can I do ?

@Fabian-Schmidt
Copy link
Owner

Fabian-Schmidt commented May 22, 2024

Can you please provide a log from the ESP device?
And also which values do you see in the Victron App on the overview page (before you connect to the device). This is the same datasource.

@terdt
Copy link
Author

terdt commented May 22, 2024

yes, of course, it is a little bit overloaded due to the amount of services which are running on the ESP

logs_womo-1_logs.txt

@Fabian-Schmidt
Copy link
Owner

yes, of course, it is a little bit overloaded due to the amount of services which are running on the ESP

logs_womo-1_logs.txt

Thanks for the log.
It contains the following message:

[W][victron_ble:266]: [xx:xx:xx:xx:xx:xx] Unsupported record type 0F

This means victron has not documented this type of device.
As I am not affiliated with Victron I only have access to public resources and to two devices which I purchased myself.
Either ask victron to update the announcement protocol definition to include this new device or we need to guess the data / protocol structure.
This would require a couple of examples and the corresponding data from the app.

Please let me know which way you want to go?

@terdt
Copy link
Author

terdt commented May 22, 2024

please see screenshot

IMG_6233

@terdt
Copy link
Author

terdt commented May 22, 2024

Do you know which document have to be updated and where could I find it ?

@Fabian-Schmidt
Copy link
Owner

The documentation is in the victron community
https://community.victronenergy.com/questions/187303/victron-bluetooth-advertising-protocol.html

See pdf in top post.

@terdt
Copy link
Author

terdt commented May 23, 2024

This document is quite old and can not be found on an official Victron page.
I assume the Victron path could be quite difficult.
Does following info can help ?
keshavdv/victron-ble#53
If not what do you need and how can I support ?

@Fabian-Schmidt
Copy link
Owner

To support the device I will create a separate branch and start writing some stub code.
Afterwards I need some verbose logs from you and the corresponding data displayed in the victron app in the overview page.
As there are only 5 values it should be relative easy.

@Fabian-Schmidt
Copy link
Owner

I added a new branch for Orion XS suport.

Can you please create a new config with minimal components and verbose level. E.g.:

external_components:
  - source: github://Fabian-Schmidt/esphome-victron_ble@OrionXS

logger:
  level: VERBOSE

To add support for the device I need a few example values from the log. There should be a message Encrypted message: .
And the corresponding value displayed in the Victron App. To identify the various states (Zustände) I need an example for each one. As none of the values can be negative 4-5 examples should be enough. It would be great if the Input and Output (Eingang und Ausgang) have a larger voltage difference (at least 0.5V) to make it easier to find them.

Thanks

@terdt
Copy link
Author

terdt commented May 26, 2024

Please find attached some data. Hopeful it is enough for evaluation. Otherwise I have to see how to log data during driving.

Victron_log.csv

@Fabian-Schmidt
Copy link
Owner

Hi,
I added support can you please test?

To ensure you get the latest version please use the following external reference:

external_components:
  - source: github://Fabian-Schmidt/esphome-victron_ble@OrionXS
    refresh: 0s

I added two additional sensor types to support the device OUTPUT_CURRENT and INPUT_CURRENT.
The device supports the following sensor types:

  • DEVICE_STATE
  • CHARGER_ERROR
  • OUTPUT_VOLTAGE
  • OUTPUT_CURRENT
  • INPUT_VOLTAGE
  • INPUT_CURRENT
  • OFF_REASON

Can you please test and let me know if it is working.

Cheers,
Fabian

@terdt
Copy link
Author

terdt commented May 27, 2024

Hi Fabian, test passed successful. Thanks a lot. Will you include as standard ? Kind Regards Thomas

@Fabian-Schmidt
Copy link
Owner

Yes I will include it as standard. In a few days I will review the code changes and then merge it.

Have you tested all the sensor types and they present correct values?

@terdt
Copy link
Author

terdt commented May 29, 2024

I have not compared all digits behind the comma, but in general all sensors reflects the values in the App. The text sensors are in English but this is also normal

@Fabian-Schmidt Fabian-Schmidt linked a pull request Jun 1, 2024 that will close this issue
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 a pull request may close this issue.

2 participants