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

Vesszővel elválasztott házszámok hibás felismerése #3558

Closed
urbalazs opened this issue Dec 12, 2023 · 6 comments
Closed

Vesszővel elválasztott házszámok hibás felismerése #3558

urbalazs opened this issue Dec 12, 2023 · 6 comments
Labels
bug Something isn't working needs testing Testing is needed before this can be closed

Comments

@urbalazs
Copy link
Collaborator

A wiki szerint:

The number may contain non-digits and if recording multiple house numbers separate them by "," (e.g. "12b,12c").

A gimmisn csak a pontosvesszővel elválasztott értékeket ismeri fel különböző házszámként:

  • a 12b,12c feldolgozás után 12-ként van felismerve
  • a 12b;12c feldolgozás után 12b-ként és 12c-ként van felismerve
@vasony
Copy link
Collaborator

vasony commented Dec 12, 2023

@urbalazs melyik területen van minta erre?
@vmiklos kéne hint nekem is, hogy a pythonban hol kell modositani

@urbalazs
Copy link
Collaborator Author

urbalazs commented Dec 12, 2023

Úgy látom, hogy csak akkor jön elő a hiba, ha:

  • Betű is van a házszám egyik tagjában.
  • Be van kapcsolva a housenumber-letters.
  • / jel is van a házszámban.

Példa a hibás felismerésre: Budapest XXII., Füttyös utca

Példa a helyes felismerésre: Budapest XXII., Naphal utca

Érdemes lenne az összes lehetőségre tesztesetet írni: 12,14, 12a,14, 12,14a, 12a,14a, 12/a,14, 12,14/a, 12/a,14/a, 12,14,16, 12a,14a,16a, 12/a,14/a,16/a.

Ha a wiki szerint vessző kell és nem pontosvessző, akkor jó lenne, ha a gimmisn se fogadná el pontosvesszővel. Akkor azonnal látnánk a hiányzó házszámoknál, ha valami pontosvesszővel lett megadva.

@urbalazs urbalazs added the bug Something isn't working label Dec 12, 2023
@vmiklos
Copy link
Owner

vmiklos commented Dec 12, 2023

A #613 óta van vessző alapon bontás, és a #112 óta van pontosvessző alapon bontás. Korábban javított issue-t nem csinálnék vissza, kivéve a szerző (itt: @vasony ) szerint már nem kell a pontosvessző.

A vessző szerint alapból próbálunk bontani, valószínűleg itt a housenumber-letters beállítással való kombinálás lesz a gond, majd igyekszem megnézni, hogy miről van szó.

@vasony a python is bont vessző mentén általában, itt csak valami corner case hibázik szerintem.

@vasony
Copy link
Collaborator

vasony commented Dec 12, 2023

Ha a wiki szerint vessző kell és nem pontosvessző, akkor jó lenne, ha a gimmisn se fogadná el pontosvesszővel. Akkor azonnal látnánk a hiányzó házszámoknál, ha valami pontosvesszővel lett megadva.

Nem a gimmisn-t használjuk QA cuccra, maradjon csak meg a pontosvessző is. A házszám ott van, értelmezhető, tehát nem hiányzik, nem kell kimenni és felmérni csak attól, hogy az általános elválasztó pontosvessző is van benne vessző helyett.
Én nem lennék ennyire szigorú, ahogy az se zavar annyira ha 40a vagy 40/A van a 40/a helyett.

Ha zavar, akkor kérdezd le overpass-szal az összeset és írd át, problem solved.

Térjünk vissza itt az eredeti problémára, a 12a,12b típusra.

vmiklos added a commit that referenced this issue Feb 4, 2024
The osm housenumber string was split by ';' already, no reason to not do
the same for ',' as well.

Addresses <#3558>.

Change-Id: Ic9766a3540f058b4bb373a9cf7a65bdf66baf909
vmiklos added a commit that referenced this issue Feb 4, 2024
The osm housenumber string was split by ';' already, no reason to not do
the same for ',' as well.

Addresses <#3558>.

Change-Id: Ic9766a3540f058b4bb373a9cf7a65bdf66baf909
@vmiklos
Copy link
Owner

vmiklos commented Feb 4, 2024

@vmiklos vmiklos added the needs testing Testing is needed before this can be closed label Feb 4, 2024
vasony added a commit that referenced this issue Feb 11, 2024
@vasony
Copy link
Collaborator

vasony commented Feb 11, 2024

Füttyös jó mindkét helyen, zárható

@vasony vasony closed this as completed Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs testing Testing is needed before this can be closed
Projects
None yet
Development

No branches or pull requests

3 participants