FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added getVertexData() and getMatrix() to PShape by codetorex · Pull Request #4373 · processing/processing · GitHub

Repository navigation

Added getVertexData() and getMatrix() to PShape - #4373

Closed
codetorex wants to merge 1 commit into
processing:masterfrom
codetorex:master
Closed

codetorex wants to merge 1 commit into
processing:masterfrom
codetorex:master

Conversation

Copy link
Copy Markdown

Hello,

In PShape class there is methods for change matrix of the shape; like translate, rotate etc. But there is no function to retrieve this matrix. Which is important for being able to copy shapes manually and create a shape from previous shape by applying some transformations. (There is static copyMatrix function but it also not public and marked as unapproved and only used by createShape that uses another shape which only does automatic copying)

Also another missing functionality was accessing raw float[] per vertex data. It seems possible to add new vertices by giving raw float array (shown in reference but I couldn't find in code, is it removed?) but is not possible retrieve it. Currently it is required to access this data by calling these functions getVertex, getTextureU getTextureV, getNormal etc. Still there is some other data (like per vertex material data) that is not accessible might be lost. It would be great if it is possible access it directly for the purpose of copying with applying different transforms.

If this changes are approved, copyGeometry function could use this getVertexData and copy the geometry by directly copying the float array.

Thanks!

diwi commented May 30, 2017

Copy link
Copy Markdown

still important.

bmFicg commented Jun 14, 2017

Copy link
Copy Markdown

Does this work with Processing ?

Copy link
Copy Markdown

@benfry, @jeremydouglass Can this be merged?

JM-FRANCE commented Nov 17, 2020 •
edited
Loading

Copy link
Copy Markdown

this is still very important. Setting also a matrix would be needed. Could this be merged in?

Copy link
Copy Markdown
Contributor

Hi @micycle1 -- sorry for delayed reply. I'm a community member, I don't actually have control over merges to this repository.

My guess would be that new features on PShape like this might be proposed as an issue / PR on Processing4 https://github.com/processing/processing4/ and ben and sam would look at it there -- but that's just a guess

Copy link
Copy Markdown

This pull request has been automatically closed and locked.

Processing 3.x is no longer actively developed. Since January 2020, development moved to the Processing 4 repository.

If you'd still like to contribute this work, please open an issue in the Processing 4 repository to discuss it first:
https://github.com/processing/processing4/issues/new/choose

Thank you for your contribution.

github-actions Bot locked as resolved and limited conversation to collaborators Jul 15, 2026
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL