Skip to content

expanded AddPitchLit to ski#6576

Merged
westnordost merged 2 commits into
streetcomplete:masterfrom
paulklie:lit_piste
Oct 18, 2025
Merged

expanded AddPitchLit to ski#6576
westnordost merged 2 commits into
streetcomplete:masterfrom
paulklie:lit_piste

Conversation

@paulklie

@paulklie paulklie commented Oct 17, 2025

Copy link
Copy Markdown
Collaborator

Fixes #6518.
Instead of implementing this as a new quest, I suggest we expand the existing IsPitchLit quest to ski pistes.
The string "Is this lit here?" already fits well.

It does not ask for piste routes, since these are filtered out (for performance reasons) anyway.

@westnordost

Copy link
Copy Markdown
Member

Uhh... I recognize there has been some prior discussion to this, but I don't see anything conclusive there.

Regarding the implementation:

  1. can you explain why piste:lit and not lit?
  2. why not update with check date + re-survey logic?

Regarding asking for it in the first place:

Does it fulfill all the quest guidelines? Can you explain it using the quest template?

@paulklie

paulklie commented Oct 17, 2025

Copy link
Copy Markdown
Collaborator Author

Regarding the implementation:

1. can you explain why `piste:lit` and not `lit`?

piste:lit is more specific for pistes. It exists since pistes can also use highways, which might have their own, independent lighting in addition to the piste lighting. I excluded any highways from this quest to ensure that we do not confuse the user or ask for the wrong one.

2. why not update with check date + re-survey logic?

piste:lit:check_date had no uses so far, so I assumed we would not want to introduce a new tag via SC, but happy to change.

Regarding asking for it in the first place:

Does it fulfill all the quest guidelines? Can you explain it using the quest template?

Aye, see #6518 (comment)

@westnordost

Copy link
Copy Markdown
Member

piste:lit is more specific for pistes. It exists since pistes can also use highways, which might have their own, independent lighting in addition to the piste lighting. I excluded any highways from this quest to ensure that we do not confuse the user or ask for the wrong one.

So? Doesn't make sense to me. Is the light emitted by piste lighting only visible to those who ski?

@westnordost

Copy link
Copy Markdown
Member

🚧 To be added tag is established and has a useful purpose
Around 16k uses at the moment.

Note the "and". So I guess the useful purpose is that it can also be used after dark? Then again, don't pistes have opening hours?

@HolgerJeromin

HolgerJeromin commented Oct 18, 2025

Copy link
Copy Markdown
Contributor

So I guess the useful purpose is that it can also be used after dark?

It is very useful. And imho correct to be piste:lit
Lit pistes are very special to have, because they can/will be part of special night skiing times.

Then again, don't pistes have opening hours?

Pistes ways are not tagged with that.
Most pistes will close after 16:00 o clock (aka last run of the lift).
The piste preparing will normally start 30 min after that (from hill down to valley).

Note: I am skiing every winter for one week since 2008.

@paulklie

paulklie commented Oct 18, 2025

Copy link
Copy Markdown
Collaborator Author

So? Doesn't make sense to me. Is the light emitted by piste lighting only visible to those who ski?

No, but it often does not operate outside of skiing season, or on certain special days. Thus it is less relevant for hikers or other users in the area.

@paulklie

paulklie commented Oct 18, 2025

Copy link
Copy Markdown
Collaborator Author

Then again, don't pistes have opening hours?

In addition to what @HolgerJeromin said: this often depends on the type of piste. nordic pistes rarely have any form of time restriction in my experience, as long as there is sufficient snow.

@HolgerJeromin

Copy link
Copy Markdown
Contributor

But perhaps the whole thing is better be done as a separate quest as in SCEE.

@westnordost

Copy link
Copy Markdown
Member

No, but it often does not operate outside of skiing season, or on certain special days. Thus it is less relevant for hikers or other users in the area.

Ok, I understand.

@westnordost

Copy link
Copy Markdown
Member

Last point: Do you think it makes sense to ask again every X years, @HolgerJeromin ?

@HolgerJeromin

Copy link
Copy Markdown
Contributor

Do you think it makes sense to ask again every X years, @HolgerJeromin ?

As this is quite complicated to add I would go for 6 years or later (or never).

@westnordost

Copy link
Copy Markdown
Member

I don't care either way, your call, @paulklie

@paulklie

paulklie commented Oct 18, 2025

Copy link
Copy Markdown
Collaborator Author

I don't care either way, your call, @paulklie

I don't think its necessary:

  1. as mentioned, piste:lit:check_date is basically unused, we would be introducing a new tag.
  2. Lighting a piste involves moving heavy equipment and kilometers of cable up a mountain, which makes me believe that it is a thing that companies rarely do, since the effort is so great. Resurveying for a rare event (adding new lighting) or an even rarer (removing all that expensive equipment) seems like not worth the effort. Adding or removing lighting from a sports pitch seems much easier / cheaper in comparison.

@westnordost westnordost merged commit c9e02f8 into streetcomplete:master Oct 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Quest: Is this piste lit?

4 participants