Repository navigation
test(draft): combined WebGL + Mono 3D compatibility validation (#294 + #296) - #300
Draft
PawelTroka wants to merge 18 commits into
Draft
PawelTroka wants to merge 18 commits into
PawelTroka wants to merge 18 commits into
Conversation
… validation and PNG export
…and export control
…and after rotation
#296) Source tree retains the exact file contents of both independently validated draft heads. This is for CI cross-platform acceptance only; do not merge without owner approval.
Scale viewport pan offsets in GetImage instead of resetting them. Capture native before/after right-button XTEST screenshots and enforce that the exported image retains the resulting pan.
…xtures Assert matched native/Web sampling coordinates and heights for paraboloid and sin/cos, and exactly 33 missing diagonal points for 1/(x-y); carry forward real-X11 pan/export regression from #296.
This was referenced Oct 27, 2026
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.
Temporary cross-platform integration validation ONLY
This is an isolated, draft-only combined candidate to verify that the independently passing WebGL 3D and native Mono 3D workstreams can coexist. Do not merge this PR or either source PR without explicit owner approval.
Exact sources
9fc1b7949fab5dab4cd3ce04cfef2bf944511b01— 21/21 CI workflows successful independently.4769084a36af7983a152367bb619aeb10c7ffaa3— 21/21 CI workflows independently successful. Its subsequent pan/export fix headdb44d1d700c6133714bed0a913d71d0b69578a28passed native X11 44/44 acceptance; full new-head matrix is separately checked.04492b3d6f0540ff0bf8623387f2eef2b9d2315e(two parents: both original heads), used for initial integrated CI.e9dc7b8d6378dc66d564a36d4681db58659dda5e: adds the exact Mono pan fix and a cross-platform 33×33 grid acceptance fixture, with an additional merge parent of Mono fixdb44d1d7. Both source PRs remain separate drafts.Acceptance questions
Important limitations
A green source branch is not integrated evidence. The numerical implementations share the expression engine and weighted grid interpolation but use different rendering/cancellation mechanisms, so visual pixel identity is neither expected nor required. Sparse valid points without renderable triangles, extremely large interval spans, macOS GUI and standalone Linux/macOS API hosting require separately scoped decisions.
This PR is a validation artifact, not a merge request.
Numerical parity fixtures (current integration head)
Matched
[-2,2]domains, 33 points/axis (WebgridSize=33, MonoQuality=20):x*x+y*y: 1,089 finite values; corner 8, center 0; verify every grid coordinate and sampled vertex.sin(x)*cos(y): 1,089 finite values; cornersin(-2)*cos(-2), center 0.1/(x-y): 1,056 finite vertices and exactly 33 missing diagonal vertices. Web asserts JSON null holes; Mono asserts native finite sample count.The GitHub Actions workflow matrix started for this updated candidate is a new exact-head run: do not reuse completed original-branch or initial integration outcomes as its result. The source PRs and #300 must remain Draft/unmerged pending owner approval.