Skip to content

Add some strip length gotchas to Help - #110

Open
OakCityDirtyRandy wants to merge 2 commits into
lukash:mainfrom
OakCityDirtyRandy:HelpStripLengthGotchas
Open

OakCityDirtyRandy wants to merge 2 commits into
lukash:mainfrom
OakCityDirtyRandy:HelpStripLengthGotchas

Conversation

@OakCityDirtyRandy

Copy link
Copy Markdown
Contributor

Some of the oddest things you'll notice on a group ride -- riding behind other VESC riders -- are weird Taillight behaviors that are often the result of misspecified strip lengths. This is especially common in daisy-chained configurations, because what you expect to see in one strip will bleed into another. This edit seeks to clarify why this happens.

@OakCityDirtyRandy

Copy link
Copy Markdown
Contributor Author

This is the revised text for the strip length help.

Note:

Strip length can be determined by counting the number of physical LEDs in the strip.
Incorrectly specified strip length will often manifest itself in animations behaving oddly.
Strip length is especially important in configurations in which the strips are daisy-chained together, which the LCM will treat as though it is a single strip.
In daisy-chained configurations, incorrect strip lengths result in animations split over multiple strips and they just don't look right.
Strip length is uniquely important for the Status bar, as it provides information on remaining charge, which might be presented incorrectly if the count is wrong.

@lukash lukash left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

An unwanted merge commit in this PR branch as well.

@lukash

lukash commented Oct 18, 2026

Copy link
Copy Markdown
Owner

Extraneous commit still on the branch.

You're adding these two bullet points:

  • Strip length can be determined by counting the number of physical LEDs in the strip.
  • Incorrectly specified strip length will often manifest itself in animations behaving oddly.

Imo the first one should be clear enough, but if you think it needs mentioning, I'd incorporate it into the first sentence something like this:

Number of LEDs in your status bar strip (you can count them as you see them).

Or something along those lines.

For the second one I'm not sure if there's anything particular which breaks animations (then it's likely a bug), I'd assume when the count is too low maybe? But I'd prefer not to have this line as it's either not useful or in case there is really odd behavior, it should be fixed.

The rest of the bullet points looks good to me.

@OakCityDirtyRandy

OakCityDirtyRandy commented Oct 18, 2026 via email

Copy link
Copy Markdown
Contributor Author

@lukash
lukash force-pushed the HelpStripLengthGotchas branch from 1cdf50f to 5d512ae Compare October 18, 2026 16:49
@lukash

lukash commented Oct 18, 2026

Copy link
Copy Markdown
Owner

I dropped the merge commit, did a fresh rebase on main and force-pushed into your branch.

@OakCityDirtyRandy

OakCityDirtyRandy commented Oct 18, 2026 via email

Copy link
Copy Markdown
Contributor Author

@lukash

lukash commented Oct 18, 2026

Copy link
Copy Markdown
Owner

I pushed a fixup commit with wording changes (I reduced the verbosity a bit again and grouped the logical points better across the bullets).

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