-
Notifications
You must be signed in to change notification settings - Fork 11
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
Update README.md #345
Update README.md #345
Conversation
* Add battery description * Absolute url * Use ros2-devel branch * Add panther_lights to url * Use ros2 branch
Merge panther_lights tables
WalkthroughThe Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
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.
Actionable comments posted: 0
Outside diff range and nitpick comments (6)
panther_lights/README.md (6)
Line range hint
124-124
: Correct verb agreement error.- It have to match one of the channels defined in the `panels` section. + It has to match one of the channels defined in the `panels` section.
Line range hint
137-137
: Add missing article "a" before "list".- The `led_animations` section contains list with definitions for various animations... + The `led_animations` section contains a list with definitions for various animations...
Line range hint
144-144
: Add missing article "the" before "id".- ...where `<ID>` is equal to `id` parameter of the given animation. + ...where `<ID>` is equal to the `id` parameter of the given animation.
Line range hint
191-191
: Add missing article "a" before "YAML".- ...user animations are parsed using YAML file and loaded on node start. + ...user animations are parsed using a YAML file and loaded on node start.
Line range hint
260-260
: Add missing article "the" before "launch".- Remember to modify launch command to use user animations: + Remember to modify the launch command to use user animations:
Line range hint
273-273
: Ensure headings are surrounded by blank lines.Please add a blank line above and below the "## Defining a Custom Animation Type" heading to improve the readability and adhere to markdown best practices.
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- panther_lights/README.md (1 hunks)
Additional context used
LanguageTool
panther_lights/README.md
[uncategorized] ~102-~102: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...o a YAML file with a description of the user defined animations. [//]: # (ROS_API_NODE_PARA...
[uncategorized] ~117-~117: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...ween multiple panels. -number_of_leds
: defines the total number of LEDs presen...
[uncategorized] ~123-~123: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ... segment has three attributes: -name
: the identifier for the segment, such as...
[uncategorized] ~124-~124: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...e between multiple segments. -channel
: This specifies which LED panel the segm...
[uncategorized] ~124-~124: This verb does not appear to agree with the subject. Consider using a different form. (AI_EN_LECTOR_REPLACEMENT_VERB_AGREEMENT)
Context: ...ch LED panel the segment belongs to. It have to match one of the channels defined in...
[uncategorized] ~125-~125: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...d in thepanels
section. -led_range
: This defines the range of LEDs within t...
[uncategorized] ~129-~129: Did you mean: “By default,”? (BY_DEFAULT_COMMA)
Context: ...ents defined in thesegments
section. By default you can use provided mapping: -all
...
[uncategorized] ~137-~137: You might be missing the article “a” here. (AI_EN_LECTOR_MISSING_DETERMINER_A)
Context: ... Theled_animations
section contains list with definitions for various animations...
[uncategorized] ~144-~144: You might be missing the article “the” here. (AI_EN_LECTOR_MISSING_DETERMINER_THE)
Context: ...MATION_**, where<ID>
is equal toid
parameter of the given animation. - ...
[uncategorized] ~176-~176: A comma might be missing here. (AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
Context: ...s]**. If animation fails to fulfill the requirement it will result in an error. #### Image...
[uncategorized] ~191-~191: You might be missing the article “a” here. (AI_EN_LECTOR_MISSING_DETERMINER_A)
Context: ... ones, user animations are parsed using YAML file and loaded on node start. For `Ima...
[uncategorized] ~260-~260: You might be missing the article “the” here. (AI_EN_LECTOR_MISSING_DETERMINER_THE)
Context: ...ty 2 and 3. Remember to modify launch command to use user animations: ``` ba...
Markdownlint
panther_lights/README.md
146-146: Expected: 2; Actual: 4 (MD007, ul-indent)
Unordered list indentation
147-147: Expected: 2; Actual: 4 (MD007, ul-indent)
Unordered list indentation
148-148: Expected: 2; Actual: 4 (MD007, ul-indent)
Unordered list indentation
273-273: Expected: 1; Actual: 0; Above (MD022, blanks-around-headings)
Headings should be surrounded by blank lines
256-256: null (MD028, no-blanks-blockquote)
Blank line inside blockquote
155-155: null (MD045, no-alt-text)
Images should have alternate text (alt text)
156-156: null (MD045, no-alt-text)
Images should have alternate text (alt text)
157-157: null (MD045, no-alt-text)
Images should have alternate text (alt text)
158-158: null (MD045, no-alt-text)
Images should have alternate text (alt text)
159-159: null (MD045, no-alt-text)
Images should have alternate text (alt text)
160-160: null (MD045, no-alt-text)
Images should have alternate text (alt text)
161-161: null (MD045, no-alt-text)
Images should have alternate text (alt text)
162-162: null (MD045, no-alt-text)
Images should have alternate text (alt text)
163-163: null (MD045, no-alt-text)
Images should have alternate text (alt text)
Additional comments not posted (1)
panther_lights/README.md (1)
269-269
: Fix syntax error in ROS2 service call command.- ros2 service call /lights/controller/set/animation panther_msgs/srv/SetLEDAnimation "{animation: {id: 0, param: ''}, repeating: true}" + ros2 service call /lights/controller/set/animation panther_msgs/srv/SetLEDAnimation "{animation: {id: 0, param: ''}, repeating: true}"Likely invalid or redundant comment.
Summary by CodeRabbit
panther_lights
package. This fixes issues with command execution due to an extra double quotation mark.