Skip to content

Optimize training images losslessly - #6

Merged
SpectrumFRC3847 merged 1 commit into
Spectrum3847:mainfrom
Project516:imgbot
Oct 15, 2026
Merged

SpectrumFRC3847 merged 1 commit into
Spectrum3847:mainfrom
Project516:imgbot

Conversation

@Project516

@Project516 Project516 commented Oct 1, 2026 •

Copy link
Copy Markdown
Member

What this changes

  • Recompresses 25 images under training/assets/ (ImgBot's automated pass).
  • Total size goes from 4,218 kB to 3,775 kB, about 10.5 percent smaller.
  • The largest savings are the three calibration snapshot PNGs (21 to 30 percent each) and logo-white.png (24 percent). The JPEGs shrink 0.4 to 18 percent.

Rebased onto main at ab6e086 on 2026-10-12. No conflicts, and the new commits since the last rebase add or change no images under training/assets/.

Checks

  • Decoded every old and new image and compared the pixels. All 25 are identical, so the reduction is lossless.
  • logo-white.png changed storage mode from RGBA to grayscale with alpha. Pixel values are the same when converted back to RGBA.
  • No code, workflow, or text files are touched. No site build was run.

@SpectrumFRC3847
SpectrumFRC3847 requested review from SpectrumFRC3847 and allengregoryiv and a balanced review from Copilot October 1, 2026 01:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

*Total -- 4,218.19kb -> 3,774.67kb (10.51%)

/training/assets/from-jetson/calibration-snapshots/calibration-snapshot-nearest_img22.png -- 431.75kb -> 302.21kb (30%)
/training/assets/logo-white.png -- 19.27kb -> 14.56kb (24.44%)
/training/assets/from-jetson/calibration-snapshots/calibration-snapshot-farthest_img15.png -- 402.73kb -> 308.14kb (23.49%)
/training/assets/from-jetson/calibration-snapshots/calibration-snapshot-most-tilted_img23.png -- 380.20kb -> 298.93kb (21.37%)
/training/assets/from-jetson/frames/tag-close_TopLeft.jpg -- 51.56kb -> 42.19kb (18.18%)
/training/assets/from-jetson/frames/no-tag_TopLeft.jpg -- 48.58kb -> 39.80kb (18.07%)
/training/assets/from-jetson/frames/tag-close-bigger_TopRight.jpg -- 59.00kb -> 49.93kb (15.37%)
/training/assets/field-images/2024-stage-51in.jpg -- 190.50kb -> 179.00kb (6.03%)
/training/assets/field-images/2026-blue-outpost-fuel.jpg -- 495.73kb -> 468.26kb (5.54%)
/training/assets/field-images/2024-speaker-143in.jpg -- 124.97kb -> 118.62kb (5.08%)
/training/assets/field-images/2024-podium-103in.jpg -- 121.40kb -> 115.27kb (5.05%)
/training/assets/field-images/2024-speaker-63in.jpg -- 138.04kb -> 131.13kb (5%)
/training/assets/field-images/2024-amp-85in.jpg -- 125.23kb -> 119.10kb (4.9%)
/training/assets/field-images/2024-speaker-hamming2.jpg -- 160.43kb -> 153.00kb (4.63%)
/training/assets/field-images/2024-speaker-amp-far.jpg -- 139.56kb -> 133.42kb (4.4%)
/training/assets/field-images/2024-speaker-source.jpg -- 139.60kb -> 134.19kb (3.87%)
/training/assets/field-images/2024-speaker-wide.jpg -- 114.32kb -> 110.00kb (3.78%)
/training/assets/photos/camera-matching.jpg -- 61.24kb -> 59.31kb (3.15%)
/training/assets/photos/pv-settings.jpg -- 59.34kb -> 57.73kb (2.71%)
/training/assets/photos/device-metrics-2cam.jpg -- 178.29kb -> 173.85kb (2.49%)
/training/assets/photos/after-123fps.jpg -- 161.37kb -> 157.70kb (2.27%)
/training/assets/photos/before-99fps.jpg -- 235.85kb -> 232.61kb (1.37%)
/training/assets/photos/bench-setup.jpg -- 180.02kb -> 177.98kb (1.13%)
/training/assets/photos/device-metrics.jpg -- 148.98kb -> 147.70kb (0.86%)
/training/assets/photos/field-3d.jpg -- 50.25kb -> 50.04kb (0.42%)

Signed-off-by: ImgBotApp <ImgBotHelp@gmail.com>
@Project516

Copy link
Copy Markdown
Member Author

@allengregoryiv ready for a review when you have a moment. Rebased onto main at ab6e086 today with no conflicts. Image-only, lossless.

@SpectrumFRC3847
SpectrumFRC3847 merged commit bf43600 into Spectrum3847:main Oct 15, 2026
1 check 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.

5 participants