Skip to content

test: add coordinate system regression demo - #1706

Merged
joeykchen merged 1 commit into
goplus:devfrom
joeykchen:refactor/spx-coordinate-system
Aug 3, 2026
Merged

test: add coordinate system regression demo#1706
joeykchen merged 1 commit into
goplus:devfrom
joeykchen:refactor/spx-coordinate-system

Conversation

@joeykchen

Copy link
Copy Markdown
Contributor

Summary

  • add a CoordinateSystem regression demo for the unified SPX/Godot coordinate conversion
  • visualize the SPX origin, axis directions, and the sprite's independent logical root
  • verify costume anchors at the geometric center and two off-center asset coordinates
  • explicitly declare the SVG source size as 120 × 80 so costume centers use the correct asset pixel space
  • exercise rotation, LeftRight flipping, movement, pen drawing, and stamping while checking logical-position invariants

Impact

The demo provides an interactive regression fixture for the logical-root/render-node split. It makes anchor conversion and render-offset regressions visible without changing runtime behavior.

Verification

cd test/CoordinateSystem
spx runnative
  • startup anchor checks: PASS
  • left/right flip round trip: PASS
  • related SPX coordinate and render-offset tests: PASS
  • JSON and SVG validation: PASS

Verified on macOS amd64 with the native Godot SPX runtime at a9b06008a.

Related

@joeykchen
joeykchen marked this pull request as ready for review August 3, 2026 08:27

@fennoai fennoai Bot left a comment

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.

No issues found in this review.

Checked the added CoordinateSystem demo files and validated the new JSON metadata. I could not run the native demo locally because the spx executable is not installed in this workspace.

@joeykchen
joeykchen merged commit 1927024 into goplus:dev Aug 3, 2026
14 checks passed
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.

1 participant