Source Control Integration - The Dark Mod Bugtracker

master 6570ea18 05.10.2021 19:45 Matthew Mott Details Diff	Matrix4::transform() now uses Vector4's Eigen structure Matrix4 was already using Eigen internally, but Vector4 wasn't. This meant that the widely-used transform() method was creating a local Eigen vector in order to perform the calculation, then converting it back into a BasicVector4. Now we simply multiply use BasicVector4's internal Eigen structure directly.
	mod - libs/math/Matrix4.h	Diff File
	mod - libs/math/Vector4.h	Diff File
master 5e906970 05.10.2021 19:31 Matthew Mott Details Diff	BasicVector4 now uses Eigen vector internally Since BasicVector4 is well encapsulated and has few references to its internal structure, relatively little code had to change.
	mod - libs/math/Vector4.h	Diff File
master 1904b34a 05.10.2021 19:14 Matthew Mott Details Diff	Remove reinterpret_cast from Vector4 Removed non-const getVector3(); this was no longer needed. The const version has been changed to return by value, eliminating the usage of reinterpret_cast entirely.
	mod - include/irenderview.h	Diff File
	mod - libs/math/Vector4.h	Diff File
	mod - libs/render/View.h	Diff File
	mod - plugins/script/interfaces/MathInterface.cpp	Diff File
	mod - test/math/Vector.cpp	Diff File
master 0b23fbc1 05.10.2021 18:51 greebo Details Diff	0005547: Recalculate the texture tool view after normalisation	Affected Issues 0005547
	mod - install/user.xml	Diff File
	mod - radiant/textool/TexTool.cpp	Diff File
	mod - radiant/textool/TexTool.h	Diff File
	mod - radiantcore/selection/textool/TextureToolSelectionSystem.cpp	Diff File
master b2efa1c1 05.10.2021 18:31 greebo Details Diff	0005547: Add ability to normalise a set of texture tool items as a whole	Affected Issues 0005547
	add - install/bitmaps/textool_normalise.png	Diff File
	mod - install/user.xml	Diff File
	mod - radiantcore/selection/textool/TextureToolSelectionSystem.cpp	Diff File
	mod - radiantcore/selection/textool/TextureToolSelectionSystem.h	Diff File
master 2eb9926a 05.10.2021 18:03 greebo Details Diff	0005547: Use the TextureNormaliser algorithm in Patch::normaliseTexture()	Affected Issues 0005547
	mod - radiantcore/patch/Patch.cpp	Diff File
	mod - test/TextureManipulation.cpp	Diff File
master ce0a9dfd 05.10.2021 17:46 greebo Details Diff	0005547: Add preliminary unit test for IPatch::normaliseTexture()	Affected Issues 0005547
	mod - test/TextureManipulation.cpp	Diff File
	mod - test/resources/tdm/maps/simple_brushes.map	Diff File
master 7c61c840 04.10.2021 17:48 greebo Details Diff	0005547: New implementation for Face::normaliseTexture(). This is now shifting the bounds origin towards the UV space origin, translating by the integer parts of the bounds center. The TextureNormaliser has been written to support multiple selected objects which are translated by the same distance.	Affected Issues 0005547
	mod - radiantcore/brush/Face.cpp	Diff File
	mod - radiantcore/selection/algorithm/Texturing.cpp	Diff File
	mod - radiantcore/selection/algorithm/Texturing.h	Diff File
	mod - test/TextureManipulation.cpp	Diff File
master 0fa0bb5f 04.10.2021 17:10 greebo Details Diff	0005547: Add unit test checking face texture normalisation	Affected Issues 0005547
	mod - test/TextureManipulation.cpp	Diff File
master b2f6ab9b 03.10.2021 19:12 greebo Details Diff	0005771: Improve the texture wrapping by using an extrapolation vector that is orthogonal to the edge center, since the source brush might be skewed.	Affected Issues 0005771
	mod - radiantcore/brush/Face.cpp	Diff File
master 59a5fc00 03.10.2021 17:55 greebo Details Diff	0005771: Simpler, but more effective algorithm, wrapping the texture of the source face around the shared edge, extrapolating it.	Affected Issues 0005771
	mod - radiantcore/brush/Face.cpp	Diff File
	mod - radiantcore/brush/TextureProjection.cpp	Diff File
	mod - radiantcore/brush/TextureProjection.h	Diff File
master 99bd25c7 03.10.2021 17:17 greebo Details Diff	0005771: Add one more unit test using a face that ends up using a 90 degree rotated texture after pasting. The current algorithm doesn't deal with that correctly, yielding an invalid texture matrix.	Affected Issues 0005771
	mod - test/TextureManipulation.cpp	Diff File
	mod - test/resources/tdm/maps/simple_brushes.map	Diff File
master f2d97567 03.10.2021 15:28 greebo Details Diff	0005771: Implement the first of several possible cases. Pasting across from z-facing to y-facing is working, and X/Y-facing sides of a cuboid are working too, but z-facing to x-facing is not.	Affected Issues 0005771
	mod - radiantcore/brush/Face.cpp	Diff File
	mod - radiantcore/selection/algorithm/Shader.cpp	Diff File
master dcbb2f29 03.10.2021 12:19 greebo Details Diff	0005771: Add unit test with the brush faces that should be textured by pasting	Affected Issues 0005771
	mod - test/Selection.cpp	Diff File
	mod - test/TextureManipulation.cpp	Diff File
	mod - test/algorithm/View.h	Diff File
	mod - test/resources/tdm/maps/simple_brushes.map	Diff File
master 601c7489 03.10.2021 05:04 greebo Details Diff	0005770: Expand unit tests to cover all three rotation commands on a defined brush: RotateSelectionX; RotateSelectionY; RotateSelectionZ	Affected Issues 0005770
	mod - test/Brush.cpp	Diff File
master c75a91bd 03.10.2021 04:35 greebo Details Diff	0005770: Change emitTextureCoordinates() behaviour to not call Face::plane3() which triggers a lot of callbacks internally. Instead, use the m_planeTransformed member which is up to date enough for all the call sites I checked. Texture coordinates are usually emitted long after the rest of the transform is calculated (in particular the normal vector), so it's not necessary to trigger some lazy transform evaluation callback just to apply the texture matrix. The method emitTextureCoordinates should do just as the name states, it's the job of the caller to make sure the normal vector is ready before emitTextureCoordinates is called.	Affected Issues 0005770
	mod - radiantcore/brush/Brush.cpp	Diff File
	mod - radiantcore/brush/Face.cpp	Diff File
	mod - radiantcore/brush/Face.h	Diff File
master 8385d906 03.10.2021 04:19 greebo Details Diff	0005770: Remove the EmitTextureCoordinates call from Face::revertTransform(), as this triggers a full brush transform evaluation through some obscure callbacks. Rather than touching this pile of traps, there'll be a separate change to fix the texture tool rotation behaviour.	Affected Issues 0005770
	mod - radiantcore/brush/Face.cpp	Diff File
master 549372e7 02.10.2021 18:21 greebo Details Diff	0005770: Craft unit test to reproduce the problem at hand	Affected Issues 0005770
	mod - test/Brush.cpp	Diff File
master 3db72214 02.10.2021 11:34 greebo Details Diff	0005746: Send a TextureChangedMessage when done with a texture tool manipulation	Affected Issues 0005746
	mod - radiantcore/selection/textool/TextureToolSelectionSystem.cpp	Diff File
master 609031fa 02.10.2021 08:40 greebo Committer: GitHub Details Diff	Merge pull request 0000024 from duzenko/ui_trivial XY context menu: Removed "..." from nested menu items	Affected Issues 0000024
	mod - radiant/ui/UserInterfaceModule.cpp	Diff File
master ddbef7d8 02.10.2021 08:38 greebo Details Diff	0005768: Swap a few buttons to reflect what the numbers are doing	Affected Issues 0005768
	mod - radiant/ui/surfaceinspector/SurfaceInspector.cpp	Diff File
master e87e9c20 02.10.2021 08:30 duzenko Details Diff	XY context menu: Removed "..." from nested menu items
	mod - radiant/ui/UserInterfaceModule.cpp	Diff File
master e14eabd4 02.10.2021 08:29 greebo Details Diff	0005768: Adjust Face::setShiftScaleRotation to handle the values emitted by the Surface Inspector input fields	Affected Issues 0005768
	mod - radiantcore/brush/Face.cpp	Diff File
	mod - radiantcore/brush/Face.h	Diff File
	mod - radiantcore/brush/TexDef.cpp	Diff File
	mod - radiantcore/brush/TexDef.h	Diff File
master d894757d 02.10.2021 07:53 greebo Details Diff	0005768: Reimplement the Face::getFakeTexCoords() method to display remotely meaningful values in the Surface Inspector	Affected Issues 0005768
	mod - radiant/ui/surfaceinspector/SurfaceInspector.cpp	Diff File
	mod - radiantcore/brush/Face.cpp	Diff File
	mod - radiantcore/map/format/primitiveparsers/BrushDef.cpp	Diff File
master cafa2928 02.10.2021 05:41 greebo Details Diff	0005734: Change SurfaceInspector left/right buttons to decrease/increase symbols	Affected Issues 0005734
	add - install/bitmaps/decrease.png	Diff File
	add - install/bitmaps/increase.png	Diff File
	mod - radiant/ui/surfaceinspector/SurfaceInspector.cpp	Diff File

Changesets: DarkRadiant