Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
490 remodel acc concept #516
490 remodel acc concept #516
Changes from 7 commits
b5d2fc0
ab7a866
1b3ea9a
5dbf85d
dbedea4
11e94fd
f55db3f
d93d240
9b3870a
6b75683
9328459
383c0e7
da0f502
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Enhance MPC section with implementation specifics
The MPC section needs more detailed information to be implementable:
Would you like me to help draft the detailed MPC implementation specifications?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me this link does not work. This might be the github preview however.
Maybe this image should also be directly included.
Especially because here some formula symbols are explained.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How can I embed an image directly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Critical: Address safety distance calculation and architectural issues
Several critical issues require immediate attention:
Proposed actions:
SafetyDistance
class with physics-based calculationsWould you like me to help draft the
SafetyDistance
class implementation?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Document safety distance calculation details
The reference to "the concept from the FLC graphic" needs explicit documentation:
This is critical for safety and should be thoroughly documented before implementation.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~77-~77: Possible missing comma found.
Context: ...sed if it is really needed. For safety distance we will use the concept from the FLC gr...
(AI_HYDRA_LEO_MISSING_COMMA)
[style] ~78-~78: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ... use the concept from the FLC graphic. For a general speed target we either take t...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[uncategorized] ~79-~79: Possible missing comma found.
Context: ...m the FLC graphic. For a general speed target we either take the speed of the car in ...
(AI_HYDRA_LEO_MISSING_COMMA)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Expand system requirements with specific metrics
The requirements section needs quantitative metrics:
Would you like me to help draft detailed requirements specifications?