Skip to content

Video decoder standalone package - #10987

Open
Pr1ncee wants to merge 11 commits into
developfrom
feat/video-decoder-standalone-package
Open

Video decoder standalone package#10987
Pr1ncee wants to merge 11 commits into
developfrom
feat/video-decoder-standalone-package

Conversation

@Pr1ncee

@Pr1ncee Pr1ncee commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Motivation and context

How has this been tested?

Checklist

  • I submit my changes into the develop branch
  • I have created a changelog fragment
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • I have linked related issues (see GitHub docs)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

Pr1ncee added 6 commits August 1, 2026 00:14
- preserve generator cleanup typing and use one chunk handle/library load\n- exercise parser, exception exports, and limited-range color conversion\n- clarify compatibility, package publication, and standalone API documentation\n- pin package test/runtime dependencies for isolated validation
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Comment thread cvat-video-openh264/LICENSE Outdated
@@ -0,0 +1,22 @@
MIT License

Copyright (C) 2018-2022 Intel Corporation

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove Intel

Comment thread cvat-video-openh264/LICENSE Outdated
MIT License

Copyright (C) 2018-2022 Intel Corporation
Copyright (C) 2022-2025 CVAT.ai Corporation

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I believe we don't put years now or just the first one. @bsekachev

Comment thread cvat-video-openh264/pyproject.toml Outdated
Comment thread cvat-video-openh264/pyproject.toml Outdated
Comment thread cvat-video-openh264/pyproject.toml Outdated
Comment thread cvat-video-openh264/README.md Outdated
Comment thread cvat-video-openh264/README.md Outdated
Comment thread cvat-video-openh264/pyproject.toml Outdated
Comment thread cvat-video-openh264/MANIFEST.in Outdated
Comment thread cvat-video-openh264/MANIFEST.in Outdated
Comment thread cvat-video-openh264/scripts/__init__.py Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What's the point of this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

it records the synthetic fixture’s provenance, license status, media properties and checksum. scripts/check_fixture_inventory.py uses it to verify that the generated test bytes remain deterministic and approved.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I understand what it does. But why? What problem does it solve?

Also, what does it mean for the test bytes to be "approved"?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

probably it's not needed and I overkilled the mechanism. I kept mp4_factory.py, it should work as well

@Pr1ncee
Pr1ncee requested a review from SpecLad August 5, 2026 19:11
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