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

Ignore access=private for all quests #607

Open
Atrate opened this issue Aug 20, 2024 · 3 comments
Open

Ignore access=private for all quests #607

Atrate opened this issue Aug 20, 2024 · 3 comments
Labels
PR welcome Needs someone to implement it

Comments

@Atrate
Copy link

Atrate commented Aug 20, 2024

Use case

I don't really get why the original SC ignores quests for things tagged access=private. In most cases it's e.g. a service road I can already see so I can map the surface and other quests without a problem and without tresspassing.

Proposed Solution

While SCEE allows to edit the overpass query for individual quests, with 216 quest types and counting it has become unfeasible to remove the access=private filters on all of them. It'd be nice if there was a toggle in the settings that did it for all of them simultaneously.

@Helium314
Copy link
Owner

It's also a lot of work for me (though it seems that less than 100 quests have a private filter), and with questionable benefit.
I'm not going do implement this, but I'm willing to accept a PR under the condition that it minimizes the chances of merge conflicts when SC filters are changed.
(I'm also not going to update it when new quests are added to SC)

@mnalis
Copy link
Collaborator

mnalis commented Aug 22, 2024

Yes, it would seem like too much work and maintenance to me too (unless it was some smart-regexp option which searched for and access ~ no|private and similar strings in element selector strings and replaced them with empty strings (but then again such generic approach might also backfire).

What I'm more worried about is that removing private restrictions should not IMHO apply to all quests. Sure, I can often see surface, sidewalk and cycleway on that access=private road, so removing private restrictions there might be useful in some cases...
But smoothness is harder (well, at least for my eyesight) - especially if that road segment is not rather short, I cannot reliably tell exact surface quality on its whole length (without walking it down)

Even more problematic are quests like drinking_water=* on that access=private well? Or internet_access=* in that access=private chalet? Or payment:* on that access=private club? I don't think can answer those (without getting in problems).

@Helium314
Copy link
Owner

I don't think can answer those (without getting in problems).

Though maybe you know the right people and can thus legally visit some of those locations...

But actually I think everything would be fine, as you have to explicitly enable the setting. So you should know that quests can be asked for places you're not allowed to access.

@Helium314 Helium314 added the PR welcome Needs someone to implement it label Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR welcome Needs someone to implement it
Projects
None yet
Development

No branches or pull requests

3 participants