Skip to content
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

LCM brightness values are passed as uint8 0..100 #2

Merged
merged 1 commit into from
Mar 24, 2024

Conversation

surfdado
Copy link

Convert brightness from 0..1 float to 0..100 integer

@lukash
Copy link
Owner

lukash commented Mar 24, 2024

As I noted in the first PR, could you please add a "Fix: Fix setting brightness for LCM modules." to the end of the commit message of the fix?

I'm trying to adopt a system for automatic changelog generation, it's a bit hairy at the moment, to my dismay. The way it works is you add a "Fix: ..." or a "Feature: ..." at the end of the commit message. It's called a git trailer, same as "Signed-off-by: " that you use. I then have a script (it's in the root of the package) that collects these and dumps them into a piece of markdown. It unfortunately doesn't handle multiline text too well 😞

Convert brightness from 0..1 float to 0..100 integer

Fix: Fix setting brightness for LCM module (Floatwheel)

Signed-off-by: Dado Mista <dadomista@gmail.com>
@lukash lukash merged commit e4500bb into lukash:main Mar 24, 2024
1 check passed
@surfdado surfdado deleted the lcmfix branch March 26, 2024 16:34
lukash pushed a commit that referenced this pull request May 31, 2024
lukash pushed a commit that referenced this pull request Sep 15, 2024
27a91f2 Merge pull request #2 from r3n33/update/profile-view
449ac43 Import colors.lisp
9b25e1e More spelling errors
eedd443 Fix spelling
55e4b53 Adding color to profile selection view

git-subtree-dir: vdisp
git-subtree-split: 27a91f26a00f7735b3b4a8910d6fdef6511ed78c
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