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

[BUG] CNG fuel status not displayed #626

Closed
3 tasks done
giovdi opened this issue May 23, 2024 · 4 comments
Closed
3 tasks done

[BUG] CNG fuel status not displayed #626

giovdi opened this issue May 23, 2024 · 4 comments
Assignees
Labels
acknowledged Bug acknowledged by the developer bug Something isn't working

Comments

@giovdi
Copy link

giovdi commented May 23, 2024

Before you post a new issue - you must first check the following (and check the boxes with an "X" below)

  • 1.) Can you login using the smartphone app Volkswagen/Volkswagen Connect and interact with your car?
  • 2.) Please make sure that you can login into https://vwid.vwgroup.io/ and make sure that any "Terms and Conditions" are all checked
  • 3.) I do not have MFA enabled.

Environment

  • Volkswagencarnet release with the issue: 4.6.3
  • Last working homeassistant-volkswagencarnet release (if known): unknown
  • Home Assistant Core release with the issue: 2024.5.4
  • Operating environment (Home Assistant OS/Supervised/Container/Core/venv): Home Assistant OS
  • Car model and year: Golf 8 2021
  • Country where the car was bought: Italy
  • Valid VW Connect subscription: yes
  • Have you verified that the We Connect service works via VW´s portal?: yes

Describe the bug

My car is dual fuel CNG+gasoline, but only the second one is shown as sensors in Home Assistant.
The range and the fuel level sensors refer to gasoline fuel.

Steps to Reproduce

You need a duel fuel CNG+gasoline car to reproduce the issue :)
Just configure it and have a look at the available sensors.

Expected behavior

It's more important to have the CNG fuel status than the gasoline.
If sensors for both primary and secondary fuels are impossible, please consider CNG as the main fuel.
In my opinion, the totalRange_km value is useless.

Screenshots

5

Traceback/Error logs

No errors are shown or reported.

Debug information - all JSON responses from logfile

Link to Pastebin: https://pastebin.com/aVcwdRkk
The paste will be deleted after the issue is closed.
The VIN is masked.

Additional context

I don't know why the configuration you suggested in the Enabling Debug Logging in Home Assistant
Wiki page
and the Enable debug logging paragraph in the README.md file didn't work for me... but I figured out with this configuration:

6
4

logger:
  default: info
  logs:
    volkswagencarnet: debug
    dashboard: debug
    custom_components.volkswagencarnet: debug
    custom_components.volkswagencarnet.climate: debug
    custom_components.volkswagencarnet.lock: debug
    custom_components.volkswagencarnet.device_tracker: debug
    custom_components.volkswagencarnet.switch: debug
    custom_components.volkswagencarnet.binary_sensor: debug
    custom_components.volkswagencarnet.sensor: debug
    volkswagencarnet.vw_connection: debug

I hope this helps! :)

Note

Issues lacking sufficient details or context may be subject to closure without prior notice. Detailed bug reports are essential for effective issue resolution. Thank you for your cooperation.

Thank you for all your work!

@giovdi giovdi added the bug Something isn't working label May 23, 2024
@stickpin
Copy link
Collaborator

Hi @giovdi,

Thanks for reporting this issue!
This is the first time I’ve encountered a vehicle with a CNG engine.
I can proceed with the development without direct access, but it would be extremely helpful if you could share your credentials. This would allow me to thoroughly test all scenarios related to the CNG vehicle.
If you’re okay with this, please also share your mobile number so I can contact you via Signal or WhatsApp before using the credentials.
You can reach me at stickpins at gmail.com.

Best Regards! :)

@stickpin stickpin added the acknowledged Bug acknowledged by the developer label May 23, 2024
@stickpin stickpin self-assigned this May 23, 2024
@giovdi
Copy link
Author

giovdi commented May 23, 2024

Hi @stickpin!

Thank you for your prompt reply :)
It's not uncommon in Italy to find a car with a CNG engine, but it's uncommon to find a VW car with a CNG engine linked to Home Assistant 😁 .

Take your time to find the best solution for this development. I'll email you ASAP with everything you need.

Best!

@stickpin
Copy link
Collaborator

stickpin commented May 23, 2024

There are some funny scenarios because of this combination of engines. :)

The vehicles I've been working on up to now were either combustion, electric, or combustion + electric.
But now, I have a scenario with two combustion engines, each with its own range and fuel/gas level. :)

I've already implemented a fix for the Combustion range sensor to consider the combined range of both combustion engines.
I've added a new Gas level sensor in addition to the existing Fuel level sensor.
Now, I'm working on adding two new sensors for vehicles with a CNG engine: Fuel range and Gas range.
All this should cover your vehicle. :)

If everything goes well, and I am able to test the implementation this evening, I will release a new beta.

stickpin added a commit that referenced this issue May 23, 2024
@stickpin
Copy link
Collaborator

Support for CNG vehicles is added in v5.0.0-beta2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
acknowledged Bug acknowledged by the developer bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants