Skip to content

add float16, int64, and uint64 - #87

Open
christian-rauch wants to merge 2 commits into
dranjan:masterfrom
christian-rauch:float16
Open

add float16, int64, and uint64#87
christian-rauch wants to merge 2 commits into
dranjan:masterfrom
christian-rauch:float16

Conversation

@christian-rauch

@christian-rauch christian-rauch commented Jul 27, 2026

Copy link
Copy Markdown

@dranjan

dranjan commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Hey @christian-rauch, thanks for the interest and the contribution. The reason those types aren't supported by plyfile is that they are not officially part of the PLY format. This project is pretty conservative about extending the PLY format (see https://python-plyfile.readthedocs.io/en/latest/developing.html#contribution-guidelines), and we would only consider it if either the proposed extension is already widespread, or if the author of the format (Greg Turk) approves of it.

This exact proposal has come up before, and it was rejected for these reasons, but I'm always willing to revisit the question given new information. I did what I did then: do a quick survey of other PLY format implementations to see what they do.

  • Greg Turk's C implementation still does not support 16-bit floats or 64-bit integers.
  • A few other implementations simply depend on or embed Diego Nehab's C implementation RPly, which also doesn't support them.
  • Of around 10 other implementations I could find, only 2 (including trimesh) support 64-bit integers, and only trimesh supports 16-bit floats. The most popular libraries do not seem to support these extensions.

Based on that, I'm not especially inclined to accept the extension at this time, but if you have more information or context to share, please do. In particular, if PLY files with these nonstandard property types are becoming more common, that would be good to know, as well as where they are coming from.

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.

2 participants