search

fansanelli / map-this-way

8 stars 4 forks source link

"Unusual value" per alcuni tipi di strade #9

Open Mannivu opened 2 years ago

Mannivu commented 2 years ago

Ho notato che c'è un avviso "Unusual value for highway" che compare quando la key highway è compilata con primary, secondary e tertiary. Tuttavia, questi sono valori standard e accettati, quindi credo che l'avviso sia da correggere.

Mannivu commented 2 years ago

Ora mi si è verificato lo stesso avviso anche per una highway=residential (questa).

canfe commented 2 years ago

Credo che sia per via del nome della strada: “vie” con la e.

Da: Manuel Tassi @.*** Inviato: mercoledì 5 ottobre 2022 19:05 A: fansanelli/map-this-way Cc: Subscribed Oggetto: Re: [fansanelli/map-this-way] "Unusual value" per alcuni tipi di strade (Issue #9)

Ora mi si è verificato lo stesso avviso anche per una highway=residential (questa https://www.openstreetmap.org/way/674061716 ).

— Reply to this email directly, view it on GitHub https://github.com/fansanelli/map-this-way/issues/9#issuecomment-1268699052 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ALMIDUTID6Q4GPNFGWV7TZDWBWYL5ANCNFSM6AAAAAAQ5XQ63M . You are receiving this because you are subscribed to this thread. https://github.com/notifications/beacon/ALMIDUWXSL2D4E6HBJBWCFDWBWYL5A5CNFSM6AAAAAAQ5XQ63OWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSLT3H2Y.gif Message ID: @.***>

Mannivu commented 2 years ago

Ah, non avevo notato quell'errore nel nome. Allora forse va affinato l'avviso?

Il gio 6 ott 2022, 19:38 canfe @.***> ha scritto:

Credo che sia per via del nome della strada: “vie” con la e.

Da: Manuel Tassi @.*** Inviato: mercoledì 5 ottobre 2022 19:05 A: fansanelli/map-this-way Cc: Subscribed Oggetto: Re: [fansanelli/map-this-way] "Unusual value" per alcuni tipi di strade (Issue #9)

Ora mi si è verificato lo stesso avviso anche per una highway=residential (questa https://www.openstreetmap.org/way/674061716 ).

— Reply to this email directly, view it on GitHub < https://github.com/fansanelli/map-this-way/issues/9#issuecomment-1268699052> , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ALMIDUTID6Q4GPNFGWV7TZDWBWYL5ANCNFSM6AAAAAAQ5XQ63M> . You are receiving this because you are subscribed to this thread. < https://github.com/notifications/beacon/ALMIDUWXSL2D4E6HBJBWCFDWBWYL5A5CNFSM6AAAAAAQ5XQ63OWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSLT3H2Y.gif> Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/fansanelli/map-this-way/issues/9#issuecomment-1270454898, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKGEKUCHFNJEAC5JIMTR4DLWB4FAXANCNFSM6AAAAAAQ5XQ63M . You are receiving this because you authored the thread.Message ID: @.***>

canfe commented 2 years ago

Si, spero che Francesco lo migliori. Per gli altri avvisi che hai trovato se me li passi posso provare a vedere se capisco il problema.

Da: Manuel Tassi @.*** Inviato: giovedì 6 ottobre 2022 19:46 A: fansanelli/map-this-way Cc: canfe; Comment Oggetto: Re: [fansanelli/map-this-way] "Unusual value" per alcuni tipi di strade (Issue #9)

Ah, non avevo notato quell'errore nel nome. Allora forse va affinato l'avviso?

Il gio 6 ott 2022, 19:38 canfe @.***> ha scritto:

Credo che sia per via del nome della strada: “vie” con la e.

Da: Manuel Tassi @.*** Inviato: mercoledì 5 ottobre 2022 19:05 A: fansanelli/map-this-way Cc: Subscribed Oggetto: Re: [fansanelli/map-this-way] "Unusual value" per alcuni tipi di strade (Issue #9)

Ora mi si è verificato lo stesso avviso anche per una highway=residential (questa https://www.openstreetmap.org/way/674061716 ).

— Reply to this email directly, view it on GitHub < https://github.com/fansanelli/map-this-way/issues/9#issuecomment-1268699052> , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ALMIDUTID6Q4GPNFGWV7TZDWBWYL5ANCNFSM6AAAAAAQ5XQ63M> . You are receiving this because you are subscribed to this thread. < https://github.com/notifications/beacon/ALMIDUWXSL2D4E6HBJBWCFDWBWYL5A5CNFSM6AAAAAAQ5XQ63OWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSLT3H2Y.gif> Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/fansanelli/map-this-way/issues/9#issuecomment-1270454898, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKGEKUCHFNJEAC5JIMTR4DLWB4FAXANCNFSM6AAAAAAQ5XQ63M . You are receiving this because you authored the thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/fansanelli/map-this-way/issues/9#issuecomment-1270463695 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ALMIDUTXJKHW4BRBPMEXDFDWB4F7LANCNFSM6AAAAAAQ5XQ63M . You are receiving this because you commented. https://github.com/notifications/beacon/ALMIDUWLHAIZ7GPSAXJ7ERTWB4F7LA5CNFSM6AAAAAAQ5XQ63OWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSLXG6M6.gif Message ID: @.***>

Mannivu commented 2 years ago

@canfe la zona è sempre quella, il paese di Arzago d'Adda in Lombardia.

fansanelli commented 2 years ago

Ciao @Mannivu confermo quanto detto da @canfe ovvero che qui:

https://github.com/fansanelli/map-this-way/blob/fca648976ac594e138608bae40281bcab01b3ba6/MapThisWay.validator.mapcss#L307

andiamo a verificare che il name specificato nella way sia un "DUG" valido. Se non compare in lista viene dato l'avviso, che cercherò di migliorare in futuro.