Skip to content

Pathing docs for yasen - #194

Open
arinemet wants to merge 5 commits into
mainfrom
pathing-docs
Open

Pathing docs for yasen#194
arinemet wants to merge 5 commits into
mainfrom
pathing-docs

Conversation

@arinemet

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Ari <nanoticity@gmail.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the src/pathing documentation to describe the pathing stack (grid BFS, spline math, and high-level path-following steps) in more detail.

Changes:

  • Added BFS pseudocode and implementation notes.
  • Added B-spline math sections (knot vector, basis function, finite differences, evaluation/derivatives).
  • Added a brief outline of the path-following approach.
Suppressed comments (3)

src/pathing/README.md:57

  • This nested bullet should be properly capitalized and refer to BFS consistently.
	- this is coming from bfs

src/pathing/README.md:71

  • Typo: "piont" → "point".
- $P_i$ is control piont at index $i$

src/pathing/README.md:77

  • Header typo: "positioin" → "position".
### Evaluate positioin

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/pathing/README.md Outdated
Comment thread src/pathing/README.md Outdated
Comment thread src/pathing/README.md Outdated
Comment thread src/pathing/README.md Outdated
Comment thread src/pathing/README.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Ari <66433127+arinemet@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (3)

src/pathing/README.md:71

  • Spelling typo in documentation: "control piont" should be "control point".
- $P_i$ is control piont at index $i$

src/pathing/README.md:57

  • This sub-bullet is informal and unclear ("this is coming from bfs"). Consider folding it into the main definition with consistent capitalization/acronyms.
- $P_i$ is control point $i$
	- this is coming from bfs

src/pathing/README.md:77

  • Section heading has a spelling typo: "positioin" should be "position".
### Evaluate positioin

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.

2 participants