docs(conductor): register PLY Binary IO and write_mesh options tracks - #27
Merged
Conversation
PLY Binary IO (#25) covers binary write support and making read_ply honor the endianness declared in the file header. These land together: read_ply currently reinterprets raw bytes as native regardless of what the header says, so a writer that can emit binary must not be able to produce files the reader refuses or silently misreads. Sequenced before ply-multichart_20260624 (#19) -- both rewrite write_ply_header and write_ply_data, multichart is Pending at 0/15, and binary IO is the live need. write_mesh Options Struct (#26) is deferred from that work and not scoped: PLYFormat lands on write_ply only, because a PLY-only value is meaningless for half of write_mesh's inputs. Registered so the intention is discoverable. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
1 participant
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.
Registers two tracks from the PLY binary IO design discussion.
ply-binary-io_20260904— PLY Binary IO (#25)Binary PLY write support, plus making
read_plyhonor the endianness declared in the file header. These are one contract and land together:read_plycurrently reinterprets raw bytes as native regardless of what the header says — native-endian reading labeled little-endian — so a writer that can emit binary must not be able to produce files the reader refuses or silently misreads.Sequenced before #19 (Multi-Chart PLY Write Support). Both rewrite
write_ply_headerandwrite_ply_data; #19 is Pending at 0/15, and binary IO is the live need —pgs-decimaterewrites its mesh once per improved search round and currently pays ASCII for it.Depends on #24.
mesh-io-options_20260904— write_mesh Options Struct (#26)Deferred from the above and deliberately unscoped.
PLYFormatlands onwrite_plyonly, because a PLY-only value is meaningless for half ofwrite_mesh's inputs and there's no good answer to whatwrite_mesh("out.obj", mesh, PLYFormat::Binary)should do. Registered so the intention is discoverable rather than living only in a design conversation.Docs only — no code changes.
🤖 Generated with Claude Code