Skip to content

Hotfix/compress img and gif#40

Merged
JaeoneLim merged 3 commits intomainfrom
hotfix/compress-img-and-gif
Feb 10, 2026
Merged

Hotfix/compress img and gif#40
JaeoneLim merged 3 commits intomainfrom
hotfix/compress-img-and-gif

Conversation

@JaeoneLim
Copy link
Copy Markdown
Contributor

지피지기 1~3편 블로그 포스트의 이미지 및 gif 압축 진행하였습니다.
1MB 이하로 압축이 안되는 gif가 있어 이는 mp4로 대체하였습니다.

Copilot AI review requested due to automatic review settings February 10, 2026 07:55
@JaeoneLim JaeoneLim requested a review from a team as a code owner February 10, 2026 07:55
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Summary of Changes

Hello @JaeoneLim, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 블로그 게시물의 전반적인 로딩 성능을 향상시키기 위해 이미지 및 GIF 파일을 최적화하는 데 중점을 둡니다. 특히, 용량이 큰 GIF 파일들을 MP4 비디오 또는 JPG 이미지로 대체하여 페이지 로딩 시간을 단축하고 사용자 경험을 개선하는 것을 목표로 합니다.

Highlights

  • 이미지 및 GIF 압축: 블로그 게시물 내 이미지 및 GIF 파일의 용량을 최적화했습니다.
  • GIF를 MP4로 대체: 1MB 이하로 압축되지 않는 GIF 파일은 MP4 비디오 형식으로 변경하여 웹 성능을 개선했습니다.
  • GIF를 JPG로 대체: 일부 GIF 파일은 정적 이미지인 JPG 형식으로 변경하여 효율성을 높였습니다.
Changelog
  • content/posts/TPU-deep-dive/index.en.md
    • systolic_array.gif를 systolic_array.mp4 비디오 임베드로 교체했습니다.
  • content/posts/TPU-deep-dive/index.ko.md
    • systolic_array.gif를 systolic_array.mp4 비디오 임베드로 교체했습니다.
  • content/posts/lpu-deep-dive/index.en.md
    • speculative-decoding.gif를 speculative-decoding-workflow.jpg 이미지로 교체했습니다.
  • content/posts/lpu-deep-dive/index.ko.md
    • speculative-decoding.gif를 speculative-decoding-workflow.jpg 이미지로 교체했습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@JaeoneLim JaeoneLim force-pushed the hotfix/compress-img-and-gif branch from 54dc95e to cc89ea7 Compare February 10, 2026 07:57
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

안녕하세요. 블로그 포스트의 이미지와 GIF 압축을 위한 변경 사항 잘 보았습니다. 용량이 큰 GIF 파일을 MP4 비디오로 대체하여 웹 페이지 로딩 성능을 개선하려는 시도는 매우 좋습니다. 다만, 비디오가 기존 GIF처럼 보이도록 몇 가지 속성을 추가하면 사용자 경험을 더욱 향상시킬 수 있습니다. 관련하여 2개의 파일에 구체적인 코드 제안을 포함한 리뷰 의견을 남겼으니 확인 부탁드립니다.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reduces media size across several blog posts by compressing images/GIFs and replacing at least one GIF with a static image (LPU deep dive) and another with an MP4 embed (TPU deep dive).

Changes:

  • Replaced the “Speculative Decoding” GIF reference in LPU deep dive posts with a JPG.
  • Replaced the “Systolic array visualization” GIF in TPU deep dive posts with an embedded MP4 video.
  • Updated/compressed multiple image assets across LPU deep dive and how-GPU-works posts.

Reviewed changes

Copilot reviewed 4 out of 47 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
content/posts/lpu-deep-dive/index.ko.md Switch speculative decoding media from GIF to JPG reference.
content/posts/lpu-deep-dive/index.en.md Switch speculative decoding media from GIF to JPG reference.
content/posts/lpu-deep-dive/images/roofline_concept.jpg Compressed/updated image asset.
content/posts/lpu-deep-dive/images/remove_ctrl_logic.png Compressed/updated image asset.
content/posts/lpu-deep-dive/images/gpu_memory_hierarchy.png Compressed/updated image asset.
content/posts/lpu-deep-dive/images/all_reduce.png Compressed/updated image asset.
content/posts/lpu-deep-dive/gifs/speculative-decoding.gif Removed/replaced (no longer referenced from the post).
content/posts/how-GPU-works/images/gpu-cuda-logo.png Compressed/updated image asset.
content/posts/how-GPU-works/images/14-thdblk-alloc.png Compressed/updated image asset.
content/posts/how-GPU-works/images/13-cuda-pm.png Compressed/updated image asset.
content/posts/TPU-deep-dive/index.ko.md Replace GIF embed with MP4 <video> embed via shortcode.
content/posts/TPU-deep-dive/index.en.md Replace GIF embed with MP4 <video> embed via shortcode.
content/posts/TPU-deep-dive/systolic_array.gif Removed/replaced (now using MP4 embed).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +64 to +69
{{< rawhtml >}}
<video autoplay loop muted playsinline width="100%" style="max-width: 100%; border-radius: 8px;">
<source src="/posts/TPU-deep-dive/systolic_array.mp4" type="video/mp4">
브라우저가 비디오 재생을 지원하지 않습니다.
</video>
{{< /rawhtml >}}
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

{{< rawhtml >}} shortcode is used here, but the repository doesn’t contain a layouts/shortcodes/rawhtml.html (and Goldmark unsafe HTML isn’t enabled in hugo.yaml). As-is, Hugo will fail to render the page or strip the <video> tag. Add the rawhtml shortcode (or enable markup.goldmark.renderer.unsafe: true and embed the <video> directly).

Suggested change
{{< rawhtml >}}
<video autoplay loop muted playsinline width="100%" style="max-width: 100%; border-radius: 8px;">
<source src="/posts/TPU-deep-dive/systolic_array.mp4" type="video/mp4">
브라우저가 비디오 재생을 지원하지 않습니다.
</video>
{{< /rawhtml >}}
<video autoplay loop muted playsinline width="100%" style="max-width: 100%; border-radius: 8px;">
<source src="/posts/TPU-deep-dive/systolic_array.mp4" type="video/mp4">
브라우저가 비디오 재생을 지원하지 않습니다.
</video>

Copilot uses AI. Check for mistakes.
![Systolic array visualization](systolic_array.gif)
{{< rawhtml >}}
<video autoplay loop muted playsinline width="100%" style="max-width: 100%; border-radius: 8px;">
<source src="/posts/TPU-deep-dive/systolic_array.mp4" type="video/mp4">
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

The <source> uses an absolute path (/posts/TPU-deep-dive/systolic_array.mp4). This is brittle if the site is ever hosted under a subpath or if Hugo’s output path casing differs; it also diverges from the relative asset links used elsewhere in the post bundle. Prefer a relative URL to the page bundle resource (e.g., systolic_array.mp4) or generate the URL via Hugo helpers/shortcode so it’s always correct.

Suggested change
<source src="/posts/TPU-deep-dive/systolic_array.mp4" type="video/mp4">
<source src="systolic_array.mp4" type="video/mp4">

Copilot uses AI. Check for mistakes.
Comment on lines +67 to +72
{{< rawhtml >}}
<video autoplay loop muted playsinline width="100%" style="max-width: 100%; border-radius: 8px;">
<source src="/posts/TPU-deep-dive/systolic_array.mp4" type="video/mp4">
Your browser does not support the video tag.
</video>
{{< /rawhtml >}}
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

{{< rawhtml >}} shortcode is used here, but the repository doesn’t contain a layouts/shortcodes/rawhtml.html (and Goldmark unsafe HTML isn’t enabled in hugo.yaml). As-is, Hugo will fail to render the page or strip the <video> tag. Add the rawhtml shortcode (or enable markup.goldmark.renderer.unsafe: true and embed the <video> directly).

Suggested change
{{< rawhtml >}}
<video autoplay loop muted playsinline width="100%" style="max-width: 100%; border-radius: 8px;">
<source src="/posts/TPU-deep-dive/systolic_array.mp4" type="video/mp4">
Your browser does not support the video tag.
</video>
{{< /rawhtml >}}
[Watch the systolic array animation (MP4)](/posts/TPU-deep-dive/systolic_array.mp4)

Copilot uses AI. Check for mistakes.
@seungbinshin
Copy link
Copy Markdown
Contributor

review

Copy link
Copy Markdown
Contributor

@seungbinshin seungbinshin left a comment

Choose a reason for hiding this comment

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

reviewed.gif

@JaeoneLim JaeoneLim merged commit 8a931ea into main Feb 10, 2026
1 check passed
@JaeoneLim JaeoneLim deleted the hotfix/compress-img-and-gif branch February 10, 2026 08:02
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.

3 participants