Skip to content

Fix: Handled 3X3 Matrix transpose with the custom input matrix - #9198

Merged
perminder-17 merged 7 commits into
processing:mainfrom
Pranava-Pai-N:bugfix/invalid-matrix-transpose-with-custom-input
Sep 24, 2026
Merged

perminder-17 merged 7 commits into
processing:mainfrom
Pranava-Pai-N:bugfix/invalid-matrix-transpose-with-custom-input

Conversation

@Pranava-Pai-N

Copy link
Copy Markdown
Contributor

Resolves #9197

Changes:

  • Fixed p5.Matrix.transpose() for 3x3 matrices when a p5.Matrix object is provided as input.
  • Converted the provided matrix to its underlying mat3 values before transposing.
  • Enabled regression tests for 3x3 matrix and array transpose behavior.
  • Kept the existing array-input behavior unchanged.

PR Checklist

  • npm run lint passes
  • [Inline reference] is included / updated
  • [Unit tests] are included / updated

@p5-bot

p5-bot Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

@perminder-17 perminder-17 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@perminder-17
perminder-17 merged commit 5442984 into processing:main Sep 24, 2026
4 checks passed
@Pranava-Pai-N

Copy link
Copy Markdown
Contributor Author

@perminder-17 Thanks for the review and the merge, hoping to contribute to more issues in the repo soon.

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.

[p5.js 2.0+ Bug Report]: 3x3 p5.Matrix.transpose() ignores passed matrix/array input and produces incorrect results

2 participants