cyberjunky / home-assistant-p2000

:fire_engine: This component tracks P2000 emergency events in The Netherlands.
MIT License
55 stars 9 forks source link

Wrong Regio name with regio code 15 #79

Open JohanEikelenboom opened 1 year ago

JohanEikelenboom commented 1 year ago

It looks like regio 15 is reported as "Brabant Noord" instead of "Haaglanden"

ArnolddeBolster commented 5 months ago

@cyberjunky Hi Ron, het probleem met lon / lat is opgelost en waarschijnlijk ook snel de verkeerde regiocode. Ik hoop dat je deze intergratie nog intact houdt naast die met de RTL stick. Voor mij werkt deze zonder de stick perfect. Dank voor deze oplossing, ben er blij mee !!!

cyberjunky commented 5 months ago

De integratie is erg simpel, problemen zijn meestal te wijten aan issues met de feed, die niet van mij is. Ik heb gekeken of ik de andoid app feed kan gebruiken, maar dan moet ik teveel omschrijven en uitzoeken.

ArnolddeBolster commented 5 months ago

@cyberjunky Hoi Ron,

Waarschijnlijk is het met de huidige feed al opgelost dus omschrijven is (voorlopig) niet nodig verwacht ik.

Mvg

Arnold


Van: Ron @.> Verzonden: woensdag 10 april 2024 15:25 Aan: cyberjunky/home-assistant-p2000 @.> CC: olybol @.>; Comment @.> Onderwerp: Re: [cyberjunky/home-assistant-p2000] Wrong Regio name with regio code 15 (Issue #79)

De integratie is erg simpel, problemen zijn meestal te wijten aan issues met de feed, die niet van mij is. Ik heb gekeken of ik de andoid app feed kan gebruiken, maar dan moet ik teveel omschrijven en uitzoeken.

— Reply to this email directly, view it on GitHubhttps://github.com/cyberjunky/home-assistant-p2000/issues/79#issuecomment-2047540226, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQ7FYPCESXTMHRI6X3PFIGDY4U4UDAVCNFSM6AAAAAAYER7SQGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBXGU2DAMRSGY. You are receiving this because you commented.Message ID: @.***>

ArnolddeBolster commented 5 months ago

@cyberjunky nog even een vraagje. Ik neem aan dat alle attributen uit de feed komen > of verrijk jij die in een script adhv andere ontvangen code ?

cyberjunky commented 5 months ago

Ik bereken alleen de afstand/distance tussen home lat/lon van HA naar lat/lon van de melding, de rest komt uit de feed.

ArnolddeBolster commented 5 months ago

@cyberjunky dat dacht ik al en wilde het zeker weten. vandaag contact gehad met de houder van de feed en herstelde daarna de lon/lat vrij snel. (oorzaak stroomstoring / oplossing herstart server) Heb gelijk gevraagd de verkeerde regionaam aan te passen en daar wordt dus nog aan gewerkt verwacht ik.

ArnolddeBolster commented 5 months ago

@cyberjunky Hoi Ron, Ik heb contact gehad met Erwin en in zijn ha databse zitten de meldingen met capcodes en heeft deze gedeeld. IN zijn database staan capcodes (Start Eind), Regio (naam) en regionaam (RegioCode)correct dus lijkt erop dat eea niet helemaal goed gaat in het verwerkingsscript. Dit zijn de code die hij heeft staan: Start Eind Regio Dienst RegioCode 0 9999 Landelijke Testdoeleinden n.v.t. 00

kan ik de lijst op een andere manier met je delen ?

Verder heb ik voor de zekerheid de volgende vraag gesteld maar wellicht is die voor jou bestemd: Nogmaals een check gedaan op de rss feed p2000.brandweer-berkel-enschot.nl/homeassistant/rss.asp en daar zie ik toch echt verkeerde regionaam staan

Kan ik daaruit concluderen dat deze feed ontwikkeld is door Ron en dat deze gebruik maakt van de feed die geel ge-arceerd staat ?

1520048 - A2 DP10 Zoetermeer Brusselstraat ZOETMR VWS 15148 1520048 A2 DP10 Zoetermeer Brusselstraat ZOETMR VWS 15148 4.48745 52.06174 15 **Brabant Noord** = GE-ARCEERDE TEKST Ambulancediensten P2000 Livemonitor 2021 HomeAssistant http://p2000.brandweer-berkel-enschot.nl/LiveMonitor.aspx = GE-ARCEERDE TEKST P2000 Thu, 11 Apr 2024 16:28:53 +0100

Als ik de yaml code goed interpreteer dan wordt de regcode gevalideerd en bij match verrijjkt met data en bij mismatch ongemoeid gelaten. Kan het zijn dat het hier fout gaat ?

Check regio

            if "regcode" in entry:
                if self._regiolist:
                    _LOGGER.debug("Filtering on Regio(s) %s", self._regiolist)
                    regiofound = False
                    for regio in self._regiolist:
                        _LOGGER.debug(
                            "Searching for regio %s in %s",
                            regio,
                            event_regio,
                        )
                        if event_regio == regio:
                            _LOGGER.debug("Regio matched")
                            regiofound = True
                            break
                        _LOGGER.debug("Regio mismatch, discarding")
                        continue
                    if not regiofound:
                        continue
notDavid commented 4 months ago

@cyberjunky dat dacht ik al en wilde het zeker weten. vandaag contact gehad met de houder van de feed en herstelde daarna de lon/lat vrij snel. (oorzaak stroomstoring / oplossing herstart server) Heb gelijk gevraagd de verkeerde regionaam aan te passen en daar wordt dus nog aan gewerkt verwacht ik.

Ter info, geen longitude/latitude te zien voor regio 24.

Een voorbeeld:

<--- details ```log longitude: 0 latitude: 0 distance: 0 capcode: 1033233 regio: 24 regio name: Limburg Zuid discipline: Politiediensten time: 2024-04-26T04:29:10 attribution: P2000 Livemonitor 2021 HomeAssistant icon: mdi:car-emergency ```

En de config:

<-- config ```lua sensor: - platform: p2000 name: landgraaf8km regios: 24 scan_interval: 30 ```
ArnolddeBolster commented 3 months ago

solved