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

POL + ITA #7643

Merged
merged 6 commits into from
Aug 20, 2024
Merged

POL + ITA #7643

merged 6 commits into from
Aug 20, 2024

Conversation

si404
Copy link
Contributor

@si404 si404 commented Aug 20, 2024

No description provided.

@si404 si404 merged commit 61c9cc6 into TravelMapping:master Aug 20, 2024
@michihdeu
Copy link
Contributor

michihdeu commented Aug 21, 2024

@si404 I don't know if I was the only traveler but do generally suggest the subsequent changes:
2024-08-20;Poland;DW680 (Ostrowek);;Route deleted -->
2024-08-20;Poland;DW680 (Ostrowek);pol.dw799;Route deleted (DW799 now)

hwy_data/POL/poldw/pol.dw799.wpt:
Wis http://www.openstreetmap.org/?lat=51.989766&lon=21.243289 ->
Wis +DW680 http://www.openstreetmap.org/?lat=51.989766&lon=21.243289

I think that the E25Cag change needs an update entry (one traveler that isn't me - well, I'll travel it in 4 weeks).

Thank you!

@si404
Copy link
Contributor Author

si404 commented Aug 21, 2024

Done the Polish change. I don't think the E25 warrants an update entry as it's merely moving a point to the correct location a short distance away

@michihdeu
Copy link
Contributor

michihdeu commented Aug 21, 2024

@si404 about SS roads:

  • SS389DirUdd and SS389 wps need to be synced (in ITA directory, didn't check ITA-SAR)
  • E25Cag + SS554 concurrency broken
  • Please check SS729's west end. It's on concurrency with SS597 now but OSM, GM and April 2024 GSV indicate it on the direct route to SS131 which is not covered by any SS road now.

I didn't check all changes though.

@michihdeu
Copy link
Contributor

Done the Polish change. I don't think the E25 warrants an update entry as it's merely moving a point to the correct location a short distance away

Welll.... I don't know. Fair enough.

@michihdeu
Copy link
Contributor

michihdeu commented Aug 21, 2024

Well, I still had mapview opened.... There is a concurrency issue at https://travelmapping.net/hb/showroute.php?r=ita.ss389&lat=40.797811&lon=9.280851&zoom=15 SS389 meets SS597 but there are also SS729 and E840. Not sure if SS597 should exist at all here.

Same with SS597Chi just further west.

@si404
Copy link
Contributor Author

si404 commented Aug 21, 2024

Can you put these on the forum. Not least as I don't believe I am meant to be the person dealing with itass (I certainly don't want to be as it=ass) and shouldn't really have done what I did.

If you are twiddling your thumbs, perhaps you could sort out Sardinia?

@michihdeu
Copy link
Contributor

michihdeu commented Aug 21, 2024

I thought you will fix want you've just introduced when fixing what I reported for Sardinia and Sicily. @rmihailucian could you deal with my ITA comments above? Maybe this month? If not, of course I could since I want a clean system when visiting the islands in September......................................................................................................................................................................................................................... btw, we hope that we can split Italy into regions soon... @yakra is currently working on some magic behind the scenes...

@rmihailucian
Copy link
Contributor

@michihdeu I will see what I can do, but I cannot guarantee that I can find time in August, as I expect a lot of things to do at my work, and on 31st of August I will also go on holiday for 2 weeeks on Sardinia :) What are your plans? Maybe we could meet there for a few hours?

@michihdeu
Copy link
Contributor

I'll copy my points over to the forum.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants