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

VGM/RPC: Rework custom colors, smoother config, better rainbow support #53

Merged
merged 18 commits into from
Jun 2, 2024

Conversation

HaxSam
Copy link
Member

@HaxSam HaxSam commented Mar 24, 2024

What's new

  • would make a smother update of rgb color to the vgm

For the reviewer

  • I've uploaded the firmware with this patch to a device and verified its functionality
  • I've confirmed the bug to be fixed / feature to be stable

@Willy-JL Willy-JL added the enhancement New enhancement or request label May 12, 2024
HaxSam added 5 commits May 12, 2024 14:08
	modified:   applications/main/momentum_app/momentum_app.h
	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_screen.c
	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_vgm.c
	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_vgm_color.c
	modified:   applications/services/rpc/rpc_gui.c
	modified:   assets/protobuf
	modified:   lib/drivers/rgb_backlight.c
	modified:   lib/momentum/momentum.h
	modified:   lib/toolbox/colors.c
	modified:   lib/toolbox/colors.h
	modified:   targets/f7/furi_hal/furi_hal_info.c

	modified:   applications/external
	modified:   applications/main/momentum_app/momentum_app.h
	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_screen.c
	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_vgm.c
	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_vgm_color.c
	modified:   applications/services/rpc/rpc_gui.c
	modified:   assets/protobuf
	modified:   lib/drivers/rgb_backlight.c
	modified:   lib/momentum/momentum.h
	modified:   lib/toolbox/colors.c
	modified:   lib/toolbox/colors.h
	modified:   targets/f7/furi_hal/furi_hal_info.c
still not done refrash rate from flipper change and is slow so the rainbow mode doesnt look good

	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_vgm_color.c
	modified:   applications/services/rpc/rpc_gui.c
	new file:   lib/momentum/rpc_rgb.c
	new file:   lib/momentum/rpc_rgb.h
	modified:   lib/toolbox/colors.c
	modified:   lib/toolbox/colors.h
	modified:   targets/f7/api_symbols.csv
	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_vgm.c
	modified:   applications/services/rpc/rpc_gui.c
	modified:   lib/momentum/momentum.h
	modified:   lib/momentum/rpc_rgb.c
	modified:   lib/momentum/rpc_rgb.h

	modified:   applications/main/momentum_app/scenes/momentum_app_scene_misc_vgm.c
	modified:   applications/services/rpc/rpc_gui.c
	modified:   lib/momentum/momentum.h
	modified:   lib/momentum/rpc_rgb.c
	modified:   lib/momentum/rpc_rgb.h
lib/momentum/rpc_rgb.c Outdated Show resolved Hide resolved
assets/protobuf Outdated Show resolved Hide resolved
lib/momentum/momentum.h Outdated Show resolved Hide resolved
@Willy-JL Willy-JL marked this pull request as ready for review June 2, 2024 07:25
@Willy-JL Willy-JL added feature New feature or request and removed enhancement New enhancement or request labels Jun 2, 2024
Copy link
Member

@Willy-JL Willy-JL left a comment

Choose a reason for hiding this comment

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

LGTM :D

@Willy-JL Willy-JL changed the title Rgb over rpc VGM/RPC: Rework custom colors, smoother config, better rainbow support Jun 2, 2024
@Willy-JL Willy-JL merged commit 2319475 into dev Jun 2, 2024
5 checks passed
@Willy-JL Willy-JL deleted the rgb_over_rpc branch June 2, 2024 07:42
@Willy-JL Willy-JL self-assigned this Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants