cgtobi / PyRMVtransport

:bus: Get transport information from opendata.rmv.de
MIT License
12 stars 6 forks source link

Example code doesn't work #52

Open petrm opened 2 years ago

petrm commented 2 years ago

The basic example provided doesn't work. The exception:

File "/home/petr/tmp/PyRMVtransport/lib/python3.10/site-packages/RMVtransport/rmvtransport.py", line 78, in get_departures raise RMVtransportError(err) from err RMVtransport.errors.RMVtransportError: no such child: JourneyList

I tried the example, because the Home Assistant integration did not work with the example configuration: 2021-12-20 17:46:32 ERROR (MainThread) [RMVtransport.rmvtransport] Can not load data from RMV API

cgtobi commented 2 years ago

Thanks for the feedback. That query fails likely because there currently are no trains going from Wiesbaden. Non the less that should be handled more gracefully.

petrm commented 2 years ago

So I tried that, changed station to 3004690 and the example now works. Interestingly, using the same with HASS still fails with the "Can not load data from RMV API". Any hints how to debug this? I have super minimal config:

cgtobi commented 2 years ago

I'll take a look shortly.

cgtobi commented 2 years ago

I can't replicate the error. Should you have an account on Discord feed free to ping me there to debug this interactively.

petrm commented 2 years ago

Thanks for having a look. I have following observation: when I set HASS to debug logging, the sensor is created for 3004690 as sensor.rmv_journey and name Obertshausen Bahnhof.

Changing logger to level goes back to the original error [RMVtransport.rmvtransport] Can not load data from RMV API.

petrm commented 2 years ago

I have done some more digging and discovered that with the minimal config above, it actually creates an "unnamed_sensor", instead of using the station name. Setting on HASS log level to debug creates the sensor with the station name as described in the docs. Still not sure how is this possible.

codesalatdev commented 3 weeks ago

Thanks for the feedback. That query fails likely because there currently are no trains going from Wiesbaden. Non the less that should be handled more gracefully.

Is there any update regarding this? I could probably contribute the improved handling of such, but I feel like a fix is definitely needed because my logs are getting spammed every weekend when a certain bus line isn't in service. The custom RMV card also does not expect this and causes cards of working schedules to fail displaying in the UI.