-
-
Notifications
You must be signed in to change notification settings - Fork 87
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
Rgb over rpc #53
base: dev
Are you sure you want to change the base?
Rgb over rpc #53
Conversation
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
…into rgb_over_rpc
@@ -0,0 +1 @@ | |||
#include "rpc_rgb.h" |
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.
are these 2 needed?
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.
Not anymore I could delete those files
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.
if this is now a fork, might need to update .gitmodules
{"White", {0xFFFF}}, | ||
{"Off", {{0, 0, 0}}}, | ||
{"Rainbow", {{0, 0, 0}}}, | ||
{"RgbMod", {{0, 0, 0}}}, |
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.
is the intent to set rainbow and rgbmod modes with the foreground and background toggles? if so, we can get rid of the other entry in the menu for "VGM Colors"
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.
Yes because you can now set the background or the foreground with rainbow and not one of them.
Wanted to ask if we should restructure or keep it like how I did it.
FuriHalVersionColor spoof_color; | ||
RgbColorTransmit vgm_color_fg; | ||
RgbColorTransmit vgm_color_bg; |
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.
changing data type here, might be good to change name too so it is not imported incorrectly. then again, also need to update the code part in settings.c, so maybe can change it to save as hex and check data length instead of saving as uint32
What's new
For the reviewer