P0: harden and consolidate the repository for archival - #1
Merged
Conversation
- rebuild landmark batches for every frame - preserve valid empty-detection results - guard model outputs, crop bounds, and pipeline status - retain computed face geometry for downstream consumers
- own detector and pose results with RAII - clamp invalid detection intervals to a safe cadence - preserve one pose result per detected person - validate polymorphic outputs and remove debug-only early exits
- pass frame dimensions in the documented width-height order - isolate flattened landmark offsets for every face - validate calculator lifecycle, inputs, and output cardinality - expose deterministic calculator teardown for native callers
- export stable native entry points across supported compilers - validate pointers, dimensions, model state, and image loading - contain C++ exceptions at Unity and WebAssembly boundaries - expose deterministic model destruction and status semantics
Som5ra
marked this pull request as ready for review
July 23, 2026 09:18
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
1 participant
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.
Summary
This PR completes the P0 archive-readiness pass on an isolated branch.
Custom/CUSTOM/customnames;Breaking change
The neutral-name migration changes native target names, C exports, WebAssembly bindings, namespaces, status types, geometry types, and examples. No compatibility aliases are retained because the archive must not preserve the former-company surface.
Downstream consumers must update their native library imports and exported-function declarations together.
Validation
The automated archive-readiness workflow validates Python/JSON syntax, repository invariants, pinned dependency bootstrap, Linux core and Unity compilation, and compilation of supported optional examples.
The final merge is gated on a green workflow and an independent re-review of the exact head.
Remaining archival decisions
Public licensing, dependency checksum/provenance policy, and large-binary handling remain owner decisions documented in
ARCHIVE.md; they do not change this P0 code-safety pass.