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

Added support for XMP streaming writes for TIFF/DNG #433

Merged
merged 31 commits into from
Mar 27, 2024
Merged

Conversation

mauricefisher64
Copy link
Collaborator

Changes in this pull request

Adds XMP streaming write support.

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.

mauricefisher64 and others added 30 commits July 27, 2022 18:58
…ff_support

* 'main' of https://github.com/contentauth/c2pa-rs: (78 commits)
  Add Exif Assertion support (#140)
  (IGNORE) Disregard grep non-zero exit status (#139)
  Prepare 0.14.0 release
  (IGNORE) Fix GH Actions logic error that is causing CI failure now (#137)
  (MINOR) Remove previously embedded manifests for remote manifests (#136)
  (MINOR) Add support for manifest removal (#123)
  Prepare 0.13.2 release
  manifest_data was missing for remote manifests (#135)
  (IGNORE) Restore cargo-semver-checks step (#129)
  Prepare 0.13.1 release
  Add ManifestStore::from_manifest_and_asset_bytes_async (#130)
  (IGNORE) Disable cargo-semver-checks step (#126)
  (IGNORE) Enforce semantic versioning requirements during PR validation and publish workflows (#124)
  Prepare 0.13.0 release
  Add RemoteManifestUrl Error, returning url (#120)
  Convert status_log error val to a string so that we can return full errors (#121)
  Report failures from remote manifest fetch (#116)
  Fast XMP extraction from PNG (#117)
  Bump MSRV to 1.59.0 (#118)
  Make sure there is  a single manifest store in the asset (#114)
  ...

# Conflicts:
#	sdk/src/error.rs
#	sdk/src/jumbf_io.rs
Clippy fixes
bug fixes
…ff_support

# Conflicts:
#	sdk/Cargo.toml
#	sdk/src/asset_handlers/mod.rs
#	sdk/src/asset_handlers/tiff_io.rs
#	sdk/src/jumbf_io.rs
#	sdk/src/store.rs
…ff_support

# Conflicts:
#	sdk/Cargo.toml
#	sdk/src/assertions/data_hash.rs
#	sdk/src/assertions/thumbnail.rs
#	sdk/src/assertions/user_cbor.rs
#	sdk/src/asset_handlers/jpeg_io.rs
#	sdk/src/ingredient.rs
#	sdk/src/store.rs
#	sdk/src/time_stamp.rs
@mauricefisher64 mauricefisher64 changed the title Added support for XMP streaming writes Added support for XMP streaming writes for TIFF/DNG Mar 27, 2024
@codecov-commenter
Copy link

codecov-commenter commented Mar 27, 2024

Codecov Report

Attention: Patch coverage is 91.66667% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 80.13%. Comparing base (b3e85c7) to head (a864329).

Files Patch % Lines
sdk/src/asset_handlers/tiff_io.rs 91.66% 5 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #433      +/-   ##
==========================================
+ Coverage   79.92%   80.13%   +0.21%     
==========================================
  Files          82       82              
  Lines       25532    25585      +53     
==========================================
+ Hits        20406    20503      +97     
+ Misses       5126     5082      -44     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mauricefisher64 mauricefisher64 merged commit 3de4152 into main Mar 27, 2024
22 checks passed
@mauricefisher64 mauricefisher64 deleted the tiff_support branch March 27, 2024 19:14
# 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.

3 participants