home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
72.72k stars 30.45k forks source link

Integration Nederlandse Spoorwegen seems to mixup 2 routes #46387

Closed groenmarsmannetje closed 3 years ago

groenmarsmannetje commented 3 years ago

The problem

I have setup the following routes:

  - name: Bodegraven-Utrecht
    from: Bdg
    to: Ut
  - name: Utrecht-Bodegraven
    from: Ut
    to: Bdg
  - name: Alphen-Utrecht
    from: Apn
    to: Ut
  - name: Utrecht-Alphen
    from: Ut
    to: Apn
  - name: Alphen-Leiden
    from: Apn
    to: Ldl
  - name: Leiden-Alphen
    from: Ldl
    to: Apn

But the last one (Leiden-Alphen) results in a sensor with the same route as the fifth route (Alphen-Leiden). It is not clear to me why this is consistently happening to this route.

What is version of Home Assistant Core has the issue?

Home Assistant 2021.2.2

What was the last working version of Home Assistant Core?

unknown

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Nederlandse Spoorwegen

Link to integration documentation on our website

https://www.home-assistant.io/integrations/nederlandse_spoorwegen/

Example YAML snippet

# Put your YAML below this line
  - platform: nederlandse_spoorwegen
    api_key: !secret ns_api_key
    routes:
      - name: Bodegraven-Utrecht
        from: Bdg
        to: Ut
      - name: Utrecht-Bodegraven
        from: Ut
        to: Bdg
      - name: Alphen-Utrecht
        from: Apn
        to: Ut
      - name: Utrecht-Alphen
        from: Ut
        to: Apn
      - name: Alphen-Leiden
        from: Apn
        to: Ldl
      - name: Leiden-Alphen
        from: Ldl
        to: Apn

Anything in the logs that might be useful for us?

# Put your logs below this line
probot-home-assistant[bot] commented 3 years ago

nederlandse_spoorwegen documentation nederlandse_spoorwegen source (message by IssueLinks)

probot-home-assistant[bot] commented 3 years ago

Hey there @yarmom, mind taking a look at this issue as its been labeled with an integration (nederlandse_spoorwegen) you are listed as a codeowner for? Thanks! (message by CodeOwnersMention)

groenmarsmannetje commented 3 years ago

I played around with this by:

But in all cases it seems that consistenly the 2 same routes are always mixed up. Maybe there is something wrong happening in the API with the combination of these 2 train stations: Apn and Ldl. Is there a way to capture the API responses?

github-actions[bot] commented 3 years ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

groenmarsmannetje commented 3 years ago

Reopen - Issue still exists