Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add write support for HEIC, HEIF, AVIF #210

Merged
merged 26 commits into from
Mar 14, 2023
Merged

Add write support for HEIC, HEIF, AVIF #210

merged 26 commits into from
Mar 14, 2023

Conversation

mauricefisher64
Copy link
Collaborator

Changes in this pull request

This PR adds support to patch offsets for formast that use /meta/iloc to reference data. Specifically we have enabled support for HEIC, HEIF, AVIF

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

@codecov-commenter
Copy link

codecov-commenter commented Mar 13, 2023

Codecov Report

Merging #210 (1175099) into main (a7d8b9e) will increase coverage by 0.10%.
The diff coverage is 79.43%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
+ Coverage   78.80%   78.91%   +0.10%     
==========================================
  Files          70       70              
  Lines       17625    17843     +218     
==========================================
+ Hits        13890    14080     +190     
- Misses       3735     3763      +28     
Impacted Files Coverage Δ
sdk/src/asset_handlers/bmff_io.rs 80.09% <62.77%> (-0.77%) ⬇️
sdk/src/store.rs 88.60% <100.00%> (+0.49%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Collaborator

@scouten-adobe scouten-adobe left a comment

Choose a reason for hiding this comment

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

A few minor comments; otherwise 👍

@mauricefisher64 mauricefisher64 merged commit 1c10b05 into main Mar 14, 2023
@mauricefisher64 mauricefisher64 deleted the RIFF branch March 14, 2023 14:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants