Font Constructor Implementation/Documentation mismatch

[erik-thompson-seattle]

There appears to be a documenation mismatch with the implementation. The documentation says the Font.DEFAULT is used if a font family is not specified, but the font family does not match. Also the size of the default argument to the constructor does not match the documentation.

https://pybricks.com/ev3-micropython/media.html#fonts

ev3 = EV3Brick()
font = Font() # expected Lucida 12
print(font.height, font.family) # got Helvetica 14
# font.DEFAULT according to the documentation is Lucida 12
print(font.DEFAULT.height, font.DEFAULT.family)  # got Lucida 23

Output from EV3

14 Helvetica
23 Lucida

Device System Info

========== ev3dev-sysinfo ==========
Image file:         ev3-micropython-v2.0.0-sd-card-image
Kernel version:     4.14.117-ev3dev-2.3.5-ev3
Brickman:           0.10.3
BogoMIPS:           148.88
Bluetooth:          
Board:              board0
BOARD_INFO_HW_REV=8
BOARD_INFO_MODEL=LEGO MINDSTORMS EV3
BOARD_INFO_ROM_REV=6
BOARD_INFO_SERIAL_NUM=0016535D0AC0
BOARD_INFO_TYPE=main

[dlech]

height is not the same thing at size. height is in pixels. size is in "points" (as in a 12 point font).

And "the default value" is not the same as Font.DEFAULT. It is up to the underlying fontconfig library to decide what font family to use when one is not specified. This can probably be changed by config files, but that is a bit beyond the scope of Pybricks.

So, it isn't really wrong, just perhaps a bit short on details.

[erik-thompson-seattle]

I see, yeah my mind connected the wrong dots, as the documentation said it works with bitmap fonts, I connected points to pixels. Keeping the two separate I can see the confusion I had.