-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
boards: arm: unify Nucleo-64 boards connectors image #15926
Conversation
All checks are passing now. Review history of this comment for details about previous failed status. |
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.
Change is ok, but please rework the (otherwise great) commit message.
Maybe you can remove the analyzis (which will not be lost, since it is here), and
keep an abstract.
At the moment there are two images for Nucleo-64 STM32 family board connectors (img/nucleo_xxxxxx_connectors.png). The images have the same canvas size (800x619) but different file size in bytes and different color coding. The first one (192K, 8-bit colormap) is used for * nucleo_f070rb * nucleo_f091rc * nucleo_f103rb * nucleo_l053r8 * nucleo_l073rz The second one (464K, 8-bit/color RGBA) is used for * nucleo_f030r8 * nucleo_f302r8 * nucleo_f334r8 Applying simultaneous black/white threshold to the images and comparing them with imagemagick tools shows that the images have no significant difference. Therefore we can use nucleo_l073rz_connectors.png for nucleo_f030r8, nucleo_f302r8 and nucleo_f334r8. Please see zephyrproject-rtos#15926 for details. Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
Please review updated commit message. |
At the moment we have different images for for Nucleo F030R8 and Nucleo F070RB boards, the images have the same pixel size but different file formats, e.g: NAMES="f030r8 f070rb" for i in $NAMES; do file boards/arm/nucleo_$i/doc/img/nucleo_$i.jpg; done boards/arm/nucleo_f030r8/doc/img/nucleo_f030r8.jpg: JPEG image data, Exif standard: [TIFF image data, little-endian, direntries=0], baseline, precision 8, 500x367, frames 3 boards/arm/nucleo_f070rb/doc/img/nucleo_f070rb.jpg: JPEG image data, JFIF standard 1.01, aspect ratio, density 1x1, segment length 16, progressive, precision 8, 500x367, frames 3 The nucleo_f030r8.jpg file is larger: for i in $NAMES; do ls -1 -sh boards/arm/nucleo_$i/doc/img/nucleo_$i.jpg; done 128K boards/arm/nucleo_f030r8/doc/img/nucleo_f030r8.jpg 40K boards/arm/nucleo_f070rb/doc/img/nucleo_f070rb.jpg Applying simultaneous black/white threshold to the images and comparing them with imagemagick tools shows that the images have no significant difference. for i in $NAMES; do convert boards/arm/nucleo_$i/doc/img/nucleo_$i.jpg \ -threshold 80% /tmp/$i.png; done compare $(for i in $NAMES; do echo -n "/tmp/$i.png "; done) \ -compose src /tmp/diff.png See also 'boards: arm: unify Nucleo-64 boards connectors image' (zephyrproject-rtos#15926). Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
At the moment we have different images for for Nucleo F030R8 and Nucleo F070RB boards, the images have the same pixel size but different file formats, e.g: NAMES="f030r8 f070rb" for i in $NAMES; do file boards/arm/nucleo_$i/doc/img/nucleo_$i.jpg; done boards/arm/nucleo_f030r8/doc/img/nucleo_f030r8.jpg: JPEG image data, Exif standard: [TIFF image data, little-endian, direntries=0], baseline, precision 8, 500x367, frames 3 boards/arm/nucleo_f070rb/doc/img/nucleo_f070rb.jpg: JPEG image data, JFIF standard 1.01, aspect ratio, density 1x1, segment length 16, progressive, precision 8, 500x367, frames 3 The nucleo_f030r8.jpg file is larger: for i in $NAMES; do ls -1 -sh boards/arm/nucleo_$i/doc/img/nucleo_$i.jpg; done 128K boards/arm/nucleo_f030r8/doc/img/nucleo_f030r8.jpg 40K boards/arm/nucleo_f070rb/doc/img/nucleo_f070rb.jpg Applying simultaneous black/white threshold to the images and comparing them with imagemagick tools shows that the images have no significant difference. for i in $NAMES; do convert boards/arm/nucleo_$i/doc/img/nucleo_$i.jpg \ -threshold 80% /tmp/$i.png; done compare $(for i in $NAMES; do echo -n "/tmp/$i.png "; done) \ -compose src /tmp/diff.png See also 'boards: arm: unify Nucleo-64 boards connectors image' (#15926). Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
At the moment there are two images for Nucleo-64 STM32 family board connectors, e.g.
The images have the same canvas size but different size in bytes and
different color coding:
Applying simultaneous black/white threshold to the images and comparing them with imagemagick tools shows that the images have no significant difference.
Final fix is: