Repository navigation
Research: ellipse and tangent path-repeat geometry (#110) - #115
Closed
madawei2699 wants to merge 2 commits into
Closed
madawei2699 wants to merge 2 commits into
madawei2699 wants to merge 2 commits into
Conversation
9 tasks
Contributor
Author
|
Research decision accepted and carried forward to #117. This PR is intentionally prototype/research evidence only; its SVG/prototype code should not become production CraftDAG surface. Closing without merge while preserving the research history here. |
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.
Research/prototype for #110
This is a bounded representation study, not a production geometry change. It adds no public ComponentPlan types or compiler behavior.
Findings
CircleRing/RadialRepeatcover circular shells and circular tangent-oriented repeats, but not ellipse perimeter geometry or partial-path repetition.Instances (plus a fixed-X assembly): 6,065 JSON bytes / 768 blocks. It is valid but bays do not rotate to local tangent.EllipseRing+ ellipse-onlyPathRepeatimplementation follow-up. Do not build genericCurvePathor CAD/spline semantics.Prototype evidence
prototypes/issue-110/artifacts/.The spike is intentionally outside core exports/schema. It uses integer bounds, deterministic nearest-voxel tangent rotation, validation, and explicit block budgets. The SVGs are geometry illustrations, not MinePilot's six-view preview captures; the report calls out remaining integration checks for support diagnostics, bounds estimates, and in-game voxel quality.
See
prototypes/issue-110/RESEARCH.mdfor the capability table, option comparison, API sketch, semantics, and limitations.Validation
No production feature is being proposed for merge in this PR; keep this PR Draft until the research decision is reviewed.