search

dawsonjon / PicoRX

Build a SDR SW/MW/LW Receiver with a Raspberry Pi Pico
Other
252 stars 32 forks source link

U8g2 integration #119

Closed mryndzionek closed 1 month ago

mryndzionek commented 1 month ago

Added u8g2 library and integrated it into our display driver, so that (hopefully) nothing breaks and migration can be performed gradually. Revamped the main/original page using the library. To build, download/clone u8g2 into external subdirectory (might be worth adding it as a submodule). Rebuilding from scratch (removing whole CMake build directory) is advised.

mryndzionek commented 1 month ago

Here is a preview of the new page: out

penfold42 commented 1 month ago

Looks great !

You should PR the screendump code. Could be handy for maintaining the doco

mryndzionek commented 1 month ago

Looks great !

You should PR the screendump code. Could be handy for maintaining the doco

BTW, could you take a look at the scrolling adjustment? It was late yesterday and I might have missed something.

penfold42 commented 1 month ago

Code looks ok and seems to work ok. I got the phone out again to check on the "fun'.

penfold42 commented 1 month ago

One other thing - I had to remove -Wall or it wouldn't compile

mryndzionek commented 1 month ago

One other thing - I had to remove -Wall or it wouldn't compile

Compiles cleanly on my side. Errors in u8g2?

mryndzionek commented 1 month ago

I added the code needed to record and decode a screendump. Here is another one :smile: cutecom

penfold42 commented 1 month ago

Yep - not your code

[ 0%] Building C object CMakeFiles/u8g2.dir/external/u8g2/csrc/mui.c.obj /home/penfold/github/PicoRX/external/u8g2/csrc/mui.c:591:16: error: 'mui_task_read_nth_selectable_field' defined but not used [-Werror=unused-function] 591 | static uint8_t mui_task_read_nth_selectable_field(mui_t *ui) | ^~~~~~~~~~ cc1: all warnings being treated as errors

MrSVCD commented 1 month ago

Is there a "better" 7 segment font? The one you are showing gives me shivers down my spine. "Better" is of course a matter of taste. My solution to my shivers would be a font that doesn't lean.

mryndzionek commented 1 month ago

Is there a "better" 7 segment font? The one you are showing gives me shivers down my spine. "Better" is of course a matter of taste. My solution to my shivers would be a font that doesn't lean.

LOL Fonts are details. I had to cobble together a font which was monospace and with specific width and height.

mryndzionek commented 1 month ago

Is this better? cutecom_000000 Don't focus on details in this PR. The main question is do we want to use U8G2.

mryndzionek commented 1 month ago

This, I think, is even better: cutecom_000001

mryndzionek commented 1 month ago

Okay, I think I've found something that behaves nicely and looks better: cutecom_000000

dawsonjon commented 1 month ago

Cool, love the new look!