Skip to content

New default antenna widget icons sbc changes - #83

Merged
henkwiedig merged 7 commits into
OpenIPC:masterfrom
DmtryJS:new-default-antenna-widget-icons
Apr 28, 2026
Merged

New default antenna widget icons sbc changes#83
henkwiedig merged 7 commits into
OpenIPC:masterfrom
DmtryJS:new-default-antenna-widget-icons

Conversation

@DmtryJS

@DmtryJS DmtryJS commented Mar 22, 2026

Copy link
Copy Markdown
Contributor

OSD Configuration Optimization for SBC Ground Station

Summary

This PR optimizes the PixelPilot OSD configuration (osd.json) for better screen real estate usage, improved signal granularity, and enhanced readability on SBC ground stations.

Changes Made

Icon Resolution Optimization
Before: 96×96px signal icons (too large, causing overflow)
After: 24×24px signal icons (matches other widget icons)
Benefit: Icons fit properly within screen bounds without clipping

RSSI Signal Granularity
Before: 5 signal levels
After: 8 signal levels
Benefit: More precise signal strength visualization for better link quality assessment

Antenna Icon Layout
Evenly distributed 6 antenna icons across the row
X coordinates: -250 to -80 (34px spacing between icons)
Alignment: First antenna icon aligned with VideoWidget column (x: -250)
Benefit: Consistent visual layout that scales with available antenna count

No Signal Indicator
Added dedicated no_signal.png icon for connection loss state
Separate from low-signal indicators
Benefit: Clear visual distinction between weak signal and no connection

vlcsnap-2026-03-22-15h51m58s372_updated

DmtryJS added 3 commits March 22, 2026 14:15
- Replace 96x96px signal icons with 24x24px for better screen fit
- Increase RSSI granularity from 5 to 8 signal levels
- Evenly distribute 6 antenna icons across x: -250 to x: -80 (34px spacing)
- Align antenna icons with VideoWidget column (x: -250)
- Add dedicated no_signal icon for connection loss state
@DmtryJS
DmtryJS marked this pull request as draft March 23, 2026 20:17
@DmtryJS

DmtryJS commented Mar 25, 2026

Copy link
Copy Markdown
Contributor Author

Add some curve for non linear rssi representation.

Tested this extensively in apartment conditions with pit mode + real-time RSSI OSD debugging. Key finding: at -90 dBm the adaptive link already drops to ~1Mbps with visible freezes — essentially the edge of usability. Near the receiver I measured -35 dBm, so I set full signal to -40 dBm for accurate representation.

Examples of representation weak and average signal

vlcsnap-2026-03-25-23h34m27s249 vlcsnap-2026-03-25-23h34m56s512 vlcsnap-2026-03-25-23h33m48s870

@DmtryJS
DmtryJS marked this pull request as ready for review March 25, 2026 20:49
@MarioFPVdev

Copy link
Copy Markdown
Member

Just for the sake of people like me that can't see tiny icons, could you please increase the icon size or maybe add a setting for anyone who wants to do it manually?

@DmtryJS

DmtryJS commented Mar 29, 2026

Copy link
Copy Markdown
Contributor Author

Just for the sake of people like me that can't see tiny icons, could you please increase the icon size or maybe add a setting for anyone who wants to do it manually?

Hi Mario! Unfortunately, I don't see how to flexibly implement an icon size setting. There are a few problems: layers have fixed coordinates, so when changing the size of one element, the others need to be moved manually as well.
But I'll try to increase these icons to 64px. Currently, they are 24px, just like the rest of the icons. The default icons were 96px, so 64px seems like a good compromise. Generally, it would be good to display a numeric RSSI value here, perhaps a combined value for all available antennas. This would be useful, alongside the bitrate.

@DmtryJS
DmtryJS marked this pull request as draft March 29, 2026 20:07
@bd4

bd4 commented Mar 30, 2026

Copy link
Copy Markdown

64px sounds good to me. I think the old icons were too big/distracting, and I agree with Mario that the new ones seem a little small at 24px.

@DmtryJS

DmtryJS commented Apr 25, 2026

Copy link
Copy Markdown
Contributor Author

Hello, changed the size but only increased to 40px, explained here
OpenIPC/PixelPilot_rk#117 (comment)

@DmtryJS
DmtryJS marked this pull request as ready for review April 25, 2026 10:47
@henkwiedig

Copy link
Copy Markdown
Collaborator

can you bump pixelpilot to the merge of your upstream pr ?
change PIXELPILOT_VERSION=185acbbb7b013da7e66b01640c8935ad8712bb9b in package/pixelpilot/pixelpilot.mk

@DmtryJS

DmtryJS commented Apr 27, 2026

Copy link
Copy Markdown
Contributor Author

can you bump pixelpilot to the merge of your upstream pr ? change PIXELPILOT_VERSION=185acbbb7b013da7e66b01640c8935ad8712bb9b in package/pixelpilot/pixelpilot.mk

Hello! Changed version

@henkwiedig
henkwiedig merged commit 4d263fa into OpenIPC:master Apr 28, 2026
6 checks passed
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.

4 participants