Odnośnie `service=driveway`, to te dodajemy zazwyczaj, gdy droga prowadzi do konkretnego obiektu.
Jak prowadzi do wielu obiektów, to zostawienie highway=service bez service=* jest OK.

Cześć,
drobna uwaga odnośnie mapowania podjazdów.

Część przed płotem (publiczna), nie powinna mieć dodanego `access=private`. Taką linię należy podzielić na min. 2 części i tą przed płotem najlepiej zostawić bez tagu `access`.

Jest to istotne np. gdy podjazdy się przecinają ze ścieżkami rowerowymi/chodnikami i trzeba nim poprowadzić na jezdnię (lub odwrotnie).

Kilka uwag, bo ile doceniam fakt, że pojawiło się indoor, ale dlaczego:

- usuwasz takie elementy jak kawałek dachu?

- usuwasz wewnętrzne ścieżki? Te powinny zostać przetagowane na highway=corridor (ew. zrobione na obszar/mulitpolygon) jeśli nie pełnią istotnej funkcji routingu.

- usuwasz tagi z POI zostawiając je surowe – to nieprawidłowe

- usuwasz POI zamiast je mergować (ctrl+shift+g) w nowy obiekt – tu akurat nie jest to jakieś krytywczne, ale warto to robić, wtedy jest lepsze podtrzymanie historii

- została zrobiona tylko część, warto chociaż dać takie info gdziekolwiek w opisie/komentarzu/uwadze

Powyższe częściowo wycofałem, oraz częściowo zmapowałem wschodnie skrzydło/budynek.

I proszę dodawaj źródła z czego korzystasz do zestawów zmian, bo jest to bardzo istotne.

Drobna uwaga odnośnie chodników tak jak tutaj: node/5582223763

Jeśli kończysz na dany moment mapować chodnik, to warto go dociągnąć do tego najbliższego skrzyżowania/jego fizycznego końca i połączyć do ulicy, bo wtedy zapewnia potencjalny routing.

Np. z punktu widzenia routingu, połączenie z tym (link niżej i wiem, że nie Ty to akurat mapowałeś) może być bardziej istotne niż prowadzenie Sportową, ale na pewno warto zmapować oba:
node/8890970971

W skrócie, żeby tak nie wisiał.

Teraz to wygląda super :)

EDIT. Zbyt szybko napisałem, oczywiście chodziło mi `oneway=yes`, ale Ty tutaj dodałeś `oneway=no`, czego kompletnie nie rozumiem.

Jeśli korzystasz z jakichś zewnętrznych źródeł, to należy je linkować (do każdej zmiany) i dobrze opisywać.
Jeśli to jest jakaś tymczasowa organizacja ruchu, to prawdopodobnie powinno być to zmapowane inaczej, bo to jest co najmniej dziwne.

Dlaczego dodajesz tak mały fragment jezdni? To niepoprawne.

Dlaczego dodajesz relację zakazu wjazdu na rondo?

relation/18786557

A oneway=no jest na rondach zbędne tak na marginesie, ale to powyższe totalnie rozwala routing.

Czy w terenie zatoczki autobusowe zyskały separatory względem głównej drogi?

Niezależnie od powyższego, ta zmiana jest błędna, bo rozwaliła wszystkie relacje autobusowe powiązane z tymi przystankami.

No i zmiany takiej wielkości obszarowo są super! :)

Zachęcam, żeby nie usuwać budynków widocznych na orto tylko tagować je jako demolished:building, żeby ktoś omyłkowo nie dodał ponownie :)

> Już skończyłem,

No nie do końca. Ponawiam prośbę. Choćby robić po jednym.

Jak nawet Ci się nie chcę myszką klikać na strzałkę, bo to cię spowalnia za bardzo, to możesz
otwierać okno wysłania przez ctrl+shift+strzałka w górę i 2 razy kliknąć enter :P

Może warto zacząć czytać komentarze pod swoimi zmianami i ich nie ignorować?

Dlaczego lądowisko zostało usunięte?

Proszę rób zmiany na mniejszym obszarze, choćby powiatami dzieląc.

Cześć,
jak coś to nie dodajemy tagu `segregated` do `highway=cycleway`, a do ścieżek zmapowanych jako 1 linia za pomocą `highway=path`.

Poprawiłem to i przy okazji ujednoliciłem okoliczne ścieżki: changeset/163546222

Proszę nie duplikuj tak płotów
To niepoprawne.

I nie będzie na Zachodnim żadnego punktu obsługi pasażerów?

Dzięki za weryfikację!

Oprócz restrykcji to dodatkowo 2 uwagi:

1. Jak już jest złączana droga w 1 linię, to warto podtrzymać przynajmniej historię jednej z nich, tu zostały usunięte obie – nie rozumiem dlaczego. Ja wybieram zazwyczaj tą starszą.

2. Podejście: usunę i niech ktoś naprawi relacje jest nieprawidłowe. Powinno to być zrobione w 1 zmianie, bo niestety aktualnie pomimo "poprawnego nierozdzielenia" jest gorzej niż było.

Ktoś w każdym momencie może pobrać stan mapy i dostanie zepsute dane.

Co do przenoszenia relacji w JOSMie to kilka kliknięć (np. plugin MoveMembership, albo nawet utilsplugin chyba się da).

Odtwarzanie ich z usuniętych dróg, to już o wiele więcej roboty i prościej wycofać zmianę i zrobić to od nowa.