feat: add CLI falsifiers and exit tables to thirteen rigging examples - #143
Merged
Conversation
Default smoke stays green. Each flag feeds degenerate input into an existing assertion, including unevaluated OBJ export and join without temp_override. Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>
6 tasks
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.
Summary
Phase 9, retrofit batch 3. Thirteen rigging / constraint / transform / mesh-attribute examples. Each flag feeds degenerate input into an existing assertion. Default smoke path is unchanged. README exit tables document every nonzero code the script already returned; no codes were renumbered.
--same-axis/--flat-source/ #141 / #142 shape. Not skip-flags, path selectors, or second witnesses. No substitutions.Parent of this branch is
main@adf9ace(v0.59.0), after squash-merge of #142 (fdaf467).armature-bend--zero-curldamped-track-aim--mutedriver-wave--flat-expr1.0;wave_scaleclosed form still assertedparent-inverse-orrery--skip-mpimatrix_parent_inverse; orbit closed form still assertedshape-key-blend--zero-blendTall.value = 0; 0.5 closed form still assertedvertex-weight-limit--skip-limitprop-origin-transform--skip-mpigltf-skin-roundtrip--no-skinsexport_skins=False; disk skin count still 1temp-override-join--no-overrideobject.joinwith notemp_overridedepsgraph-export--unevaluatedapply_modifiers=False; export still must equal evaluatedattribute-domain-shear--no-overwritecolor-attribute-wheel--point-domainsoccer-ball-goldberg--invert-binddepsgraph-export--objis a path selector and was left alone.--unevaluatedcallswm.obj_export(..., apply_modifiers=False). The depsgraph still measures the SUBSURF mesh; the OBJ is the 8-vert cage; existingexported != eval_vcountreturns 5.temp-override-join--no-overridecallsbpy.ops.object.join()with no fabricated context. ARuntimeErrorfrompoll()is caught so it is not FATAL 1;check()then sees three remaining mesh objects and returns 3. The assertion is the existing object-count gate.catalog.jsonwas not modified. Falsifiers are not on the default smoke path.Live-run table
Binaries (each
--versionfrom that exe):.scratch/blender-4.5.11-windows-x64/blender.exe4db51e9d1e1e.scratch/blender-5.1.2-windows-x64/blender.exeec6e62d40fa9.scratch/blender-5.2.1-windows-x64/blender.exe9e2066aef7efCheck-only (no
--output). Observed exit codes were identical on all three binaries.armature-bend--zero-curldamped-track-aim--mutedriver-wave--flat-exprparent-inverse-orrery--skip-mpishape-key-blend--zero-blendvertex-weight-limit--skip-limitprop-origin-transform--skip-mpigltf-skin-roundtrip--no-skinstemp-override-join--no-overridedepsgraph-export--unevaluatedattribute-domain-shear--no-overwritecolor-attribute-wheel--point-domainsoccer-ball-goldberg--invert-bindLive-run-proven: the 78 headless invocations (13 × 2 × 3). No falsifier exited 0. No default exited nonzero.
5.1 evidence in this PR body is local, from
.scratch/blender-5.1.2-windows-x64/blender.exe. CI 5.1 is theneeds-5.1smoke job on this PR, not this table.Inspection-only: README table wording vs
return Nin each script; remaining-count arithmetic (Phase 6's 41 − 6 − 10 − 13 = 12). Did not restyle prior batches orbake-normal-high-to-low. Did not touch pathology/sidecar five,catalog.json,VERSION,CHANGELOG.md,release.yml,blender-smoke.yml,pages.yml, gallery assets.Test plan
needs-5.1Remaining no-falsifier pool
12 of 55 — the Phase 10 / batch 4 set.