depth-preserving raw output in P3D needs depth sorting #2235

Closed
codeanticode opened this Issue Nov 24, 2013 · 3 comments

Comments

Projects
None yet
3 participants
@codeanticode
Member

codeanticode commented Nov 24, 2013

Otherwise, the triangles are renderer in the order they were defined in the sketch code, which leads to incorrect occlusions, see for example:

http://forum.processing.org/two/discussion/1509/3d-scene-output-to-pdf-depth-buffer-issues

The triangle sorting algorithm can be used when rendering semitransparent geometry: #90

@JakubValtar

This comment has been minimized.

Show comment
Hide comment
@JakubValtar

JakubValtar Jul 16, 2015

Contributor

Original code example works great in current HEAD when you export with ENABLE_DEPTH_SORT hint. Should we sort by default when people export to PDF from P3D, or should we expect people to enable the sorter themselves?

Contributor

JakubValtar commented Jul 16, 2015

Original code example works great in current HEAD when you export with ENABLE_DEPTH_SORT hint. Should we sort by default when people export to PDF from P3D, or should we expect people to enable the sorter themselves?

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Jul 28, 2015

Member

Need a processing-docs issue here to update the PDF and beginRaw() reference to tell people to use hint(ENABLE_DEPTH_SORT) with 2D renderers.

Member

benfry commented Jul 28, 2015

Need a processing-docs issue here to update the PDF and beginRaw() reference to tell people to use hint(ENABLE_DEPTH_SORT) with 2D renderers.

@alignedleft alignedleft referenced this issue in processing/processing-docs Jul 29, 2015

Closed

Update PDF and beginRaw() reference pages #271

@codeanticode

This comment has been minimized.

Show comment
Hide comment
@codeanticode

codeanticode Aug 3, 2015

Member

The functionality is already implemented, closing now since the documentation update is addressed in the processing-docs issue referenced above.

Member

codeanticode commented Aug 3, 2015

The functionality is already implemented, closing now since the documentation update is addressed in the processing-docs issue referenced above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment