Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 10, 2025

Corrects typos and grammatical errors across README, notebooks, and Python comments.

Changes:

  • README.md: "you should your ROS" → "you should see your ROS"
  • pid_controller_node.py: "contoller" → "controller"
  • PID_heading_controller.ipynb:
    • "trail and error" → "trial and error"
    • "where the decision making part" → "the decision making part"
    • "deigned" → "designed"
    • "decrease the the time" → "decrease the time"
    • "fields ref and and v_0" → "fields ref and v_0"
    • "two consecutive instant" → "two consecutive instants"
    • "Derivates" → "Derivatives"
    • "derivates" → "derivatives"
    • "infinitesimaly" → "infinitesimally"
    • "signals that helps" → "signals that help" (subject-verb agreement)
    • "you browser" → "your browser"
    • "What we can do, is" → "What we can do is" (removed incorrect comma)
  • PID_offset_controller.ipynb:
    • "fedelity" → "fidelity"
    • "funtion" → "function"
    • "fields ref and and v_0" → "fields ref and v_0"
    • "you browser" → "your browser"

Created from VS Code via the GitHub Pull Request extension.

Original prompt

Fix any spelling or grammatical errors.

Created from VS Code via the GitHub Pull Request extension.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix spelling and grammatical errors Fix spelling and grammatical errors in documentation and code comments Nov 10, 2025
@liammcalpineduckietown
Copy link
Contributor

@copilot Can you find any more?

Copy link
Contributor Author

Copilot AI commented Nov 10, 2025

@copilot Can you find any more?

Found and fixed 3 additional errors:

  • "decrease the the time" → "decrease the time" in PID_heading_controller.ipynb
  • "fields ref and and v_0" → "fields ref and v_0" in both notebooks (2 occurrences)

Fixed in commit 5de6c98.

@liammcalpineduckietown
Copy link
Contributor

@copilot Can you find any more?

Copy link
Contributor Author

Copilot AI commented Nov 10, 2025

@copilot Can you find any more?

Found and fixed 4 additional spelling errors in PID_heading_controller.ipynb:

  • "two consecutive instant" → "two consecutive instants"
  • "Derivates" → "Derivatives"
  • "derivates" → "derivatives"
  • "infinitesimaly" → "infinitesimally"

Fixed in commit 2ce6dd2.

@liammcalpineduckietown
Copy link
Contributor

@copilot Can you find any more?

Co-authored-by: liammcalpineduckietown <[email protected]>
@liammcalpineduckietown
Copy link
Contributor

@copilot Try again.

@liammcalpineduckietown
Copy link
Contributor

@copilot Try again.

Copy link
Contributor Author

Copilot AI commented Nov 10, 2025

@copilot Try again.

Found and fixed 1 subject-verb agreement error:

  • "signals that helps" → "signals that help" in PID_heading_controller.ipynb

Fixed in commit 85a63c6.

@liammcalpineduckietown
Copy link
Contributor

@copilot Can you find any more?

Copy link
Contributor Author

Copilot AI commented Nov 10, 2025

@copilot Can you find any more?

Found and fixed 2 pronoun errors:

  • "you browser" → "your browser" in both notebooks (2 occurrences)

Fixed in commit 9f24e2f.

@liammcalpineduckietown
Copy link
Contributor

@copilot Can you find any more?

Copy link
Contributor Author

Copilot AI commented Nov 10, 2025

@copilot Can you find any more?

Found and fixed 1 punctuation error:

  • "What we can do, is" → "What we can do is" (removed incorrect comma in PID_heading_controller.ipynb)

Fixed in commit 234bcc0.

@liammcalpineduckietown
Copy link
Contributor

@copilot Can you find any more?

Copy link
Contributor Author

Copilot AI commented Nov 10, 2025

@copilot Can you find any more?

I've conducted an extensive search using multiple techniques including pattern matching for common typos, grammar errors, subject-verb agreement issues, punctuation problems, and manual review of all text content. No additional spelling or grammatical errors were found. The repository documentation appears to be clean of errors at this point.

@liammcalpineduckietown liammcalpineduckietown marked this pull request as ready for review November 10, 2025 13:35
@liammcalpineduckietown liammcalpineduckietown requested review from tanij and removed request for liammcalpineduckietown November 10, 2025 13:36
@liammcalpineduckietown liammcalpineduckietown changed the title Fix spelling and grammatical errors in documentation and code comments DTSW-7396 Fix spelling and grammatical errors in documentation and code comments Nov 10, 2025
Copy link

@dt-test-1 dt-test-1 left a comment

Choose a reason for hiding this comment

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

image

Things look good, but please verify locally that the codes work before merging

@liammcalpineduckietown liammcalpineduckietown merged commit 260bd35 into ente Nov 26, 2025
2 checks 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.

4 participants