X Tutup
Skip to content

Fix Basis.determinant() doc: uniform scale determinant is scale^3#110424

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
Dark-Horizons:patch-1
Sep 19, 2025
Merged

Fix Basis.determinant() doc: uniform scale determinant is scale^3#110424
Repiteo merged 1 commit intogodotengine:masterfrom
Dark-Horizons:patch-1

Conversation

@Dark-Horizons
Copy link
Contributor

@Dark-Horizons Dark-Horizons commented Sep 11, 2025

Corrected a documentation mistake in Basis.determinant(). For a uniform scale basis, the determinant is scale^3, not scale^2.

…t scale^2

Corrected a documentation mistake in Basis.determinant(). For a uniform scale basis, the determinant is scale^3, not scale^2.
@Dark-Horizons Dark-Horizons requested a review from a team as a code owner September 11, 2025 15:47
@AThousandShips AThousandShips added bug documentation cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release labels Sep 12, 2025
@AThousandShips AThousandShips added this to the 4.6 milestone Sep 12, 2025
@Repiteo Repiteo merged commit 3632e6f into godotengine:master Sep 19, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Sep 19, 2025

Thanks! Congratulations on your first merged contribution! 🎉

@Repiteo
Copy link
Contributor

Repiteo commented Sep 22, 2025

Cherry-picked to 4.5

@Repiteo Repiteo removed the cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release label Sep 22, 2025
@Repiteo
Copy link
Contributor

Repiteo commented Sep 22, 2025

Cherry-picked to 4.4

@Repiteo Repiteo removed the cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release label Sep 22, 2025
@akien-mga akien-mga changed the title Fix Basis.determinant() doc: uniform scale determinant is scale^3 Fix Basis.determinant() doc: uniform scale determinant is scale^3 Jan 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

X Tutup