Skip to content

Moana first attempt - #117

Merged
doubleailes merged 5 commits into
mainfrom
moana_first_attempt
Jul 27, 2026
Merged

Moana first attempt#117
doubleailes merged 5 commits into
mainfrom
moana_first_attempt

Conversation

@doubleailes

Copy link
Copy Markdown
Owner

No description provided.

doubleailes and others added 5 commits July 27, 2026 15:37
…lytically

Replaces per-primitive Box<dyn Prim> heap allocation with a closed PrimNode
enum stored inline in one contiguous Vec, and replaces flattening each
cubic curve span into 8 straight segments with direct recursive Bezier
intersection (ported from pbrt-v4, still deferring to the exact
rounded-cone test at the leaf so round-tube fidelity is unchanged).

Dense xgen-style archives (grass, needles) previously needed millions of
individually heap-allocated segments; together these cuts let the
complete, unmodified Moana Island scene (every element, zero pruning) fit
in ~126GB instead of an extrapolated 300GB+.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@doubleailes
doubleailes merged commit f1ece22 into main Jul 27, 2026
2 checks passed
@doubleailes
doubleailes deleted the moana_first_attempt branch July 27, 2026 19:28
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