website: Convert remote MP4 embeds to local AVIF files - #407
Open
aidanfnv wants to merge 2 commits into
Open
Conversation
Contributor
Greptile SummaryConverts remote MP4 model-card embeds into locally hosted AVIF assets.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains. Important Files Changed
Reviews (2): Last reviewed commit: "website: Reduce use of raw HTML for AVIF..." | Re-trigger Greptile |
Collaborator
Author
|
/ok to test f0b3648 |
Collaborator
Author
|
/ok to test 4a0d1f1 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Currently we rely on MP4 embeds for most of the model card videos on the FlashDreams website.
This change swaps out the embeds with AVIF files made by downloading and converting those MP4s, so that if one of those links is changed or is otherwise unreachable, the model card videos will still remain undisturbed on the FlashDreams site.
The SANA-WM model cards are already using AVIF, and these will work the same.