Skip to content

Implement windward-leeward racing course#1

Open
cuccel wants to merge 5 commits into
mpihlak:masterfrom
cuccel:feature/new-course
Open

Implement windward-leeward racing course#1
cuccel wants to merge 5 commits into
mpihlak:masterfrom
cuccel:feature/new-course

Conversation

@cuccel

@cuccel cuccel commented Oct 4, 2025

Copy link
Copy Markdown

No description provided.

@mpihlak

mpihlak commented Oct 5, 2025

Copy link
Copy Markdown
Owner

Tee rebase praeguse seisu peale, lisasin eile stardiabi ja start-line biase ja võib olla veel midagi. Need tekitavad konflikte. Dashboardi asukoha võiks ka samaks jätta ja stardiliini pikkuse ja layouti, lisada ainult täiendav raja layout.
Eri rajatüüpide high-score jaoks peb vist ka midagi ette võtma.

@mpihlak

mpihlak commented Oct 5, 2025

Copy link
Copy Markdown
Owner

@claude review the pull request

@claude

claude Bot commented Oct 5, 2025

Copy link
Copy Markdown

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@mpihlak

mpihlak commented Oct 5, 2025

Copy link
Copy Markdown
Owner

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@cuccel Ma arvan et see / seal branchi nimes ajab Claude segadusse.

…h course

- WindwardFinish course: 400m line (200m each side from center) - original setup
- Pin at WorldWidth/2-200, Committee at WorldWidth/2+200
- KJK course: keeps 0.2nm line with committee boat in middle
- Finish line properly configured for each course type
- No separate finish mark for windward-finish (uses starting line)
@cuccel cuccel force-pushed the feature/new-course branch from e02d122 to 2d80f61 Compare October 6, 2025 07:16
@cuccel

cuccel commented Oct 6, 2025

Copy link
Copy Markdown
Author
  • nihutasin dashboardi tagasi (pikemad tekstid ei mahtunud ära - seega algsest asukohast pisut vasakul siiski)
  • lisasin uue raja koos "KJK-stiilis" stardi-finišiliiniga
  • muutsin "default" stardi-finišiliini tagasi.

@cuccel

cuccel commented Oct 6, 2025

Copy link
Copy Markdown
Author

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@cuccel Ma arvan et see / seal branchi nimes ajab Claude segadusse.

uurisin ta käest järele:

No, I won't get confused by branch names containing "/". Git branch names with slashes like fork/feature/new-course are completely valid and commonly used for:

Organizing branches: feature/new-feature, bugfix/issue-123, hotfix/critical-bug
Remote tracking: origin/main, fork/development
Hierarchical naming: team/frontend/new-ui, release/v2.1/final
Git treats the slash as just another character in the branch name - it's not a directory separator in this context.

pigem on probleem selles, et action käib käesolevas repos ja siin repos tõesti feature/new-course puudub. kuna mul ei ole siia reposse õigusi siis forkisin ja tegin PR'i läbi forki.

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