Skip to content

fix navigation scoring: consider number of visited landmarks - #89

Merged
Copposhop merged 5 commits into
mainfrom
patch_2026_navigation_visited_landmarks
May 6, 2026
Merged

fix navigation scoring: consider number of visited landmarks#89
Copposhop merged 5 commits into
mainfrom
patch_2026_navigation_visited_landmarks

Conversation

@max-weisser

Copy link
Copy Markdown
Member

No description provided.

@max-weisser max-weisser self-assigned this May 4, 2026
@Copposhop

Copy link
Copy Markdown
Member

To be honest, I would replace the

However, a pass is only counted if the previously required landmarks have already been visited.

in the Sceneario section with a better definition of a "sucessfully visited landmark" or similar.
Then we can just refer to the amount of sucessfully visited landmarks in the scoring section and avoid the redundant definition over there.

Clarified what a successfull pass of a landmark means in the event execution section.
Clarified that mapping and navigation happen on same track.
Removed redundant successfull passing from scoring chapter.
Added Change Highlight.:
@MariusMogk

Copy link
Copy Markdown
Contributor

To be honest, I would replace the

However, a pass is only counted if the previously required landmarks have already been visited.

in the Sceneario section with a better definition of a "sucessfully visited landmark" or similar. Then we can just refer to the amount of sucessfully visited landmarks in the scoring section and avoid the redundant definition over there.

Clarified what a successfull pass means, removed redundant phrase.
Pls review.

@Copposhop

Copy link
Copy Markdown
Member

I tried to simplify the section about the landmark sequence/order without much redundancy or overly specific definitions.
There is now a clearer definition for a sucessfully visited landmark to distinguish it from simply passing a landmark.

@Copposhop
Copposhop merged commit 2534bfd into main May 6, 2026
1 check passed
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