Merge branch 'model-factory-fixup'

19 files changed, 193 insertions, 91 deletions

diff --git a/assetFactory/cuboid.cpp b/assetFactory/cuboid.cpp
index 24fe4a4..f200258 100644
--- a/assetFactory/cuboid.cpp
+++ b/assetFactory/cuboid.cpp
@@ -2,7 +2,7 @@
 #include "modelFactoryMesh.h"
 
 Cuboid::CreatedFaces
-Cuboid::createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const
+Cuboid::createMesh(ModelFactoryMesh & mesh, float) const
 {
 	static constexpr std::array<glm::vec3, 8> VERTICES {{
 			// bottom
@@ -17,13 +17,13 @@ Cuboid::createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) c
 			{n, n, o},
 	}};
 
-	const auto vhs = addMutatedToMesh(mesh, VERTICES, mutation);
+	const auto vhs = addToMesh(mesh, VERTICES);
 	return {
-			mesh.add_namedFace("top", {vhs[4], vhs[5], vhs[6], vhs[7]}),
-			mesh.add_namedFace("bottom", {vhs[0], vhs[1], vhs[2], vhs[3]}),
-			mesh.add_namedFace("left", {vhs[0], vhs[7], vhs[6], vhs[1]}),
-			mesh.add_namedFace("right", {vhs[2], vhs[5], vhs[4], vhs[3]}),
-			mesh.add_namedFace("front", {vhs[0], vhs[3], vhs[4], vhs[7]}),
-			mesh.add_namedFace("back", {vhs[2], vhs[1], vhs[6], vhs[5]}),
+			mesh.add_namedFace("top", vhs[4], vhs[5], vhs[6], vhs[7]),
+			mesh.add_namedFace("bottom", vhs[0], vhs[1], vhs[2], vhs[3]),
+			mesh.add_namedFace("left", vhs[0], vhs[7], vhs[6], vhs[1]),
+			mesh.add_namedFace("right", vhs[2], vhs[5], vhs[4], vhs[3]),
+			mesh.add_namedFace("front", vhs[0], vhs[3], vhs[4], vhs[7]),
+			mesh.add_namedFace("back", vhs[2], vhs[1], vhs[6], vhs[5]),
 	};
 }
diff --git a/assetFactory/cuboid.h b/assetFactory/cuboid.h
index 5a4072a..4e375a9 100644
--- a/assetFactory/cuboid.h
+++ b/assetFactory/cuboid.h
@@ -4,5 +4,5 @@
 
 class Cuboid : public Shape {
 public:
-	CreatedFaces createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const override;
+	CreatedFaces createMesh(ModelFactoryMesh & mesh, float lodf) const override;
 };
diff --git a/assetFactory/cylinder.cpp b/assetFactory/cylinder.cpp
index cf0dbfb..0803369 100644
--- a/assetFactory/cylinder.cpp
+++ b/assetFactory/cylinder.cpp
@@ -3,44 +3,51 @@
 #include "modelFactoryMesh.h"
 
 Cylinder::CreatedFaces
-Cylinder::createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const
+Cylinder::createMesh(ModelFactoryMesh & mesh, float lodf) const
 {
-	const glm::vec2 scale {std::accumulate(&mutation[0][0], &mutation[0][3], 0.f),
-			std::accumulate(&mutation[1][0], &mutation[1][3], 0.f)};
-	const unsigned int P = static_cast<unsigned int>(std::round(15.F * std::sqrt(glm::length(scale))));
-	std::vector<OpenMesh::VertexHandle> bottom(P), top(P);
-	std::generate_n(bottom.begin(), P, [a = 0.f, step = two_pi / static_cast<float>(P), &mesh, &mutation]() mutable {
-		const auto xy = sincosf(a += step) * .5F;
-		const auto xyz = (xy ^ 0) % mutation;
-		return mesh.add_vertex({xyz.x, xyz.y, xyz.z});
-	});
-	std::generate_n(top.begin(), P, [a = 0.f, step = two_pi / static_cast<float>(P), &mesh, &mutation]() mutable {
-		const auto xy = sincosf(a -= step) * .5F;
-		const auto xyz = (xy ^ 1) % mutation;
-		return mesh.add_vertex({xyz.x, xyz.y, xyz.z});
+	const auto P = static_cast<unsigned int>(std::round(15.F * std::sqrt(lodf)));
+	const auto step = two_pi / static_cast<float>(P);
+
+	// Generate 2D circumference points
+	std::vector<glm::vec2> circumference(P);
+	std::generate(circumference.begin(), circumference.end(), [a = 0.f, step]() mutable {
+		return sincosf(a += step) * .5F;
 	});
+
 	CreatedFaces surface;
-	std::generate_n(std::inserter(surface, surface.end()), P,
-			[a = 0.f, step = two_pi / static_cast<float>(P), &mesh, &mutation]() mutable {
-				const auto xy1 = sincosf(a) * .5F;
-				const auto xy2 = sincosf(a -= step) * .5F;
-				const auto xyz1b = (xy1 ^ 0) % mutation;
-				const auto xyz2b = (xy2 ^ 0) % mutation;
-				const auto xyz1t = (xy1 ^ 1) % mutation;
-				const auto xyz2t = (xy2 ^ 1) % mutation;
-				return mesh.add_namedFace("edge",
-						{
-								mesh.add_vertex({xyz1b.x, xyz1b.y, xyz1b.z}),
-								mesh.add_vertex({xyz2b.x, xyz2b.y, xyz2b.z}),
-								mesh.add_vertex({xyz2t.x, xyz2t.y, xyz2t.z}),
-								mesh.add_vertex({xyz1t.x, xyz1t.y, xyz1t.z}),
-						});
-			});
-	for (const auto & [name, face] : surface) {
-		mesh.property(mesh.smoothFaceProperty, face) = true;
+	{
+		// Generate bottom face vertices
+		std::vector<OpenMesh::VertexHandle> bottom(P);
+		std::transform(circumference.begin(), circumference.end(), bottom.begin(), [&mesh](const auto & xy) {
+			return mesh.add_vertex({xy.x, xy.y, 0.f});
+		});
+		surface.insert(mesh.add_namedFace("bottom", bottom));
+	}
+	{
+		// Generate top face vertices
+		std::vector<OpenMesh::VertexHandle> top(P);
+		std::transform(circumference.rbegin(), circumference.rend(), top.begin(), [&mesh](const auto & xy) {
+			return mesh.add_vertex({xy.x, xy.y, 1.f});
+		});
+		surface.insert(mesh.add_namedFace("top", top));
+	}
+	{
+		// Generate edge vertices
+		std::vector<std::pair<OpenMesh::VertexHandle, OpenMesh::VertexHandle>> edge(P + 1);
+		std::transform(circumference.begin(), circumference.end(), edge.begin(), [&mesh](const auto & xy) {
+			return std::make_pair(mesh.add_vertex({xy.x, xy.y, 0.f}), mesh.add_vertex({xy.x, xy.y, 1.f}));
+		});
+		// Wrap around
+		edge.back() = edge.front();
+		// Transform adjacent pairs of top/bottom pairs to faces
+		std::adjacent_find(edge.begin(), edge.end(), [&mesh, &surface](const auto & first, const auto & second) {
+			const auto fh
+					= surface.insert(mesh.add_namedFace("edge", first.first, first.second, second.second, second.first))
+							  ->second;
+			mesh.property(mesh.smoothFaceProperty, fh) = true;
+			return false;
+		});
 	}
-	surface.insert(mesh.add_namedFace("bottom", bottom));
-	surface.insert(mesh.add_namedFace("top", top));
 
 	return surface;
 }
diff --git a/assetFactory/cylinder.h b/assetFactory/cylinder.h
index 65ca5e5..1d7b0e0 100644
--- a/assetFactory/cylinder.h
+++ b/assetFactory/cylinder.h
@@ -4,5 +4,5 @@
 
 class Cylinder : public Shape {
 public:
-	CreatedFaces createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const override;
+	CreatedFaces createMesh(ModelFactoryMesh & mesh, float lodf) const override;
 };
diff --git a/assetFactory/factoryMesh.cpp b/assetFactory/factoryMesh.cpp
index 0cfed85..1665b90 100644
--- a/assetFactory/factoryMesh.cpp
+++ b/assetFactory/factoryMesh.cpp
@@ -11,7 +11,7 @@ FactoryMesh::createMesh() const
 
 	ModelFactoryMesh mesh;
 	for (const auto & use : uses) {
-		use->createMesh(mesh, glm::identity<Mutation::Matrix>());
+		use->createMesh(mesh, 1);
 	}
 	mesh.garbage_collection();
 
diff --git a/assetFactory/modelFactoryMesh.cpp b/assetFactory/modelFactoryMesh.cpp
index f9ee6a0..806ac3b 100644
--- a/assetFactory/modelFactoryMesh.cpp
+++ b/assetFactory/modelFactoryMesh.cpp
@@ -5,11 +5,3 @@ ModelFactoryMesh::ModelFactoryMesh()
 	add_property(smoothFaceProperty);
 	add_property(nameFaceProperty);
 }
-
-std::pair<std::string, OpenMesh::FaceHandle>
-ModelFactoryMesh::add_namedFace(std::string name, std::vector<OpenMesh::VertexHandle> p)
-{
-	const auto handle = add_face(std::move(p));
-	property(nameFaceProperty, handle) = name;
-	return std::make_pair(name, handle);
-}
diff --git a/assetFactory/modelFactoryMesh.h b/assetFactory/modelFactoryMesh.h
index 258913b..ea5f011 100644
--- a/assetFactory/modelFactoryMesh.h
+++ b/assetFactory/modelFactoryMesh.h
@@ -35,5 +35,12 @@ struct ModelFactoryMesh : public OpenMesh::PolyMesh_ArrayKernelT<ModelFactoryTra
 	OpenMesh::FPropHandleT<bool> smoothFaceProperty;
 	OpenMesh::FPropHandleT<std::string> nameFaceProperty;
 
-	std::pair<std::string, OpenMesh::FaceHandle> add_namedFace(std::string name, std::vector<OpenMesh::VertexHandle> p);
+	template<typename... Vs>
+	std::pair<std::string, OpenMesh::FaceHandle>
+	add_namedFace(std::string name, Vs &&... vs)
+	{
+		const auto handle = add_face(std::forward<Vs>(vs)...);
+		property(nameFaceProperty, handle) = name;
+		return std::make_pair(name, handle);
+	}
 };
diff --git a/assetFactory/mutation.cpp b/assetFactory/mutation.cpp
index 21d2a24..9722dc3 100644
--- a/assetFactory/mutation.cpp
+++ b/assetFactory/mutation.cpp
@@ -1,4 +1,5 @@
 #include "mutation.h"
+#include <algorithm>
 #include <glm/gtx/transform.hpp>
 #include <maths.h>
 
@@ -8,3 +9,21 @@ Mutation::getMatrix() const
 	return glm::translate(glm::identity<Matrix>(), position) * rotate_ypr(rotation)
 			* glm::scale(glm::identity<Matrix>(), scale);
 }
+
+Mutation::Matrix
+Mutation::getDeformationMatrix() const
+{
+	return glm::scale(glm::identity<Matrix>(), scale);
+}
+
+Mutation::Matrix
+Mutation::getLocationMatrix() const
+{
+	return glm::translate(glm::identity<Matrix>(), position) * rotate_ypr(rotation);
+}
+
+float
+Mutation::relativeLevelOfDetail() const
+{
+	return std::max({scale.x, scale.y, scale.z});
+}
diff --git a/assetFactory/mutation.h b/assetFactory/mutation.h
index 440fab0..e620955 100644
--- a/assetFactory/mutation.h
+++ b/assetFactory/mutation.h
@@ -7,6 +7,10 @@ struct Mutation {
 	using Matrix = glm::mat4;
 
 	Matrix getMatrix() const;
+	Matrix getDeformationMatrix() const;
+	Matrix getLocationMatrix() const;
+
+	float relativeLevelOfDetail() const;
 
 	glm::vec3 position {};
 	glm::vec3 rotation {};
diff --git a/assetFactory/object.cpp b/assetFactory/object.cpp
index ae5a301..5c86f47 100644
--- a/assetFactory/object.cpp
+++ b/assetFactory/object.cpp
@@ -4,11 +4,11 @@
 Object::Object(std::string i) : id {std::move(i)} { }
 
 Object::CreatedFaces
-Object::createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const
+Object::createMesh(ModelFactoryMesh & mesh, float levelOfDetailFactor) const
 {
 	CreatedFaces faces;
 	for (const auto & use : uses) {
-		auto useFaces = use->createMesh(mesh, mutation);
+		auto useFaces = use->createMesh(mesh, levelOfDetailFactor);
 		std::transform(useFaces.begin(), useFaces.end(), std::inserter(faces, faces.end()), [this](auto && face) {
 			return std::make_pair(id + ":" + face.first, std::move(face.second));
 		});
diff --git a/assetFactory/object.h b/assetFactory/object.h
index f3726c7..3f7b640 100644
--- a/assetFactory/object.h
+++ b/assetFactory/object.h
@@ -10,7 +10,7 @@ public:
 	Object() = default;
 	Object(std::string i);
 
-	CreatedFaces createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const override;
+	CreatedFaces createMesh(ModelFactoryMesh & mesh, float lodf) const override;
 
 	Use::Collection uses;
 	std::string id;
diff --git a/assetFactory/plane.cpp b/assetFactory/plane.cpp
index 563c4e9..c6e1b5a 100644
--- a/assetFactory/plane.cpp
+++ b/assetFactory/plane.cpp
@@ -2,7 +2,7 @@
 #include "modelFactoryMesh.h"
 
 Plane::CreatedFaces
-Plane::createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const
+Plane::createMesh(ModelFactoryMesh & mesh, float) const
 {
 	static constexpr std::array<glm::vec3, 4> VERTICES {{
 			{n, n, z},
@@ -11,5 +11,5 @@ Plane::createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) co
 			{n, y, z},
 	}};
 
-	return {mesh.add_namedFace("plane", addMutatedToMesh(mesh, VERTICES, mutation))};
+	return {mesh.add_namedFace("plane", addToMesh(mesh, VERTICES))};
 }
diff --git a/assetFactory/plane.h b/assetFactory/plane.h
index 5e93ee4..f24e2c1 100644
--- a/assetFactory/plane.h
+++ b/assetFactory/plane.h
@@ -4,5 +4,5 @@
 
 class Plane : public Shape {
 public:
-	CreatedFaces createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const override;
+	CreatedFaces createMesh(ModelFactoryMesh & mesh, float lodf) const override;
 };
diff --git a/assetFactory/shape.cpp b/assetFactory/shape.cpp
index f6e55e8..ec6594e 100644
--- a/assetFactory/shape.cpp
+++ b/assetFactory/shape.cpp
@@ -5,12 +5,10 @@
 #include "shape.h"
 
 std::vector<OpenMesh::VertexHandle>
-Shape::addMutatedToMesh(
-		ModelFactoryMesh & mesh, const std::span<const glm::vec3> vertices, const Mutation::Matrix & mutation)
+Shape::addToMesh(ModelFactoryMesh & mesh, const std::span<const glm::vec3> vertices)
 {
 	std::vector<OpenMesh::VertexHandle> vhs;
-	std::transform(vertices.begin(), vertices.end(), std::back_inserter(vhs), [&mesh, &mutation](const auto & v) {
-		const auto p = v % mutation;
+	std::transform(vertices.begin(), vertices.end(), std::back_inserter(vhs), [&mesh](const auto & p) {
 		return mesh.add_vertex({p.x, p.y, p.z});
 	});
 	return vhs;
diff --git a/assetFactory/shape.h b/assetFactory/shape.h
index 5a2b59c..fe517c1 100644
--- a/assetFactory/shape.h
+++ b/assetFactory/shape.h
@@ -1,9 +1,9 @@
 #pragma once
 
 #include "modelFactoryMesh_fwd.h"
-#include "mutation.h"
 #include "stdTypeDefs.hpp"
 #include <OpenMesh/Core/Mesh/Handles.hh>
+#include <glm/vec3.hpp>
 #include <map>
 #include <span>
 #include <string>
@@ -18,8 +18,8 @@ public:
 
 	virtual ~Shape() = default;
 
-	virtual CreatedFaces createMesh(ModelFactoryMesh &, const Mutation::Matrix & mutation) const = 0;
+	virtual CreatedFaces createMesh(ModelFactoryMesh &, float levelOfDetailFactor) const = 0;
 
-	static std::vector<OpenMesh::VertexHandle> addMutatedToMesh(
-			ModelFactoryMesh & mesh, const std::span<const glm::vec3> vertices, const Mutation::Matrix & mutation);
+	static std::vector<OpenMesh::VertexHandle> addToMesh(
+			ModelFactoryMesh & mesh, const std::span<const glm::vec3> vertices);
 };
diff --git a/assetFactory/use.cpp b/assetFactory/use.cpp
index 708e310..898b736 100644
--- a/assetFactory/use.cpp
+++ b/assetFactory/use.cpp
@@ -1,14 +1,36 @@
 #include "use.h"
 #include "assetFactory.h"
+#include <glm/ext/matrix_transform.hpp>
+#include <glm/matrix.hpp>
+#include <maths.h>
+#include <set>
+#include <stream_support.hpp>
 
 Shape::CreatedFaces
-Use::createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const
+Use::createMesh(ModelFactoryMesh & mesh, float levelOfDetailFactor) const
 {
-	auto faces = type->createMesh(mesh, mutation * getMatrix());
+	auto apply = [&mesh](const auto & faces, const Mutation::Matrix & m) {
+		std::set<ModelFactoryMesh::VertexHandle> vs;
+		for (const auto & f : faces) {
+			const auto fvr = mesh.fv_range(f.second);
+			for (const auto & v : fvr) {
+				if (!vs.contains(v)) {
+					glm::vec3 p = mesh.point(v);
+					p = p % m;
+					mesh.set_point(v, ModelFactoryMesh::Point(p.x, p.y, p.z));
+					vs.insert(v);
+				}
+			}
+		}
+	};
+
+	auto faces = type->createMesh(mesh, levelOfDetailFactor * relativeLevelOfDetail());
 	applyStyle(mesh, {this}, faces);
+	apply(faces, getDeformationMatrix());
 	for (const auto & [name, faceController] : faceControllers) {
 		faceController->apply(mesh, {this}, name, faces);
 	}
+	apply(faces, getLocationMatrix());
 	return faces;
 }
 
diff --git a/assetFactory/use.h b/assetFactory/use.h
index 5e4c35f..9dbf382 100644
--- a/assetFactory/use.h
+++ b/assetFactory/use.h
@@ -11,7 +11,7 @@ class Use : public StdTypeDefs<Use>, public Mutation, public Style, public Persi
 public:
 	using FaceControllers = std::map<std::string, std::unique_ptr<FaceController>>;
 
-	Shape::CreatedFaces createMesh(ModelFactoryMesh & mesh, const Mutation::Matrix & mutation) const;
+	Shape::CreatedFaces createMesh(ModelFactoryMesh & mesh, float lodf) const;
 
 	Shape::CPtr type;
 	FaceControllers faceControllers;
diff --git a/res/brush47.xml b/res/brush47.xml
index 9c7d2a8..dbd3327 100644
--- a/res/brush47.xml
+++ b/res/brush47.xml
@@ -1,36 +1,89 @@
 <?xml version="1.0"?>
 <ilt p.id="assetFactory">
   <object id="wheel">
-    <use type="cylinder" position="0,0,0.571" scale="1.142,1.142,0.07" rotation="0,0,1.5708"/>
+    <use type="cylinder" position="0,0,0.571" scale="1.142,1.142,0.07" rotation="0,0,1.5708" colour="silver">
+      <face id="top" colour="#413b3a"/>
+      <face id="bottom" colour="#413b3a"/>
+    </use>
+    <use type="cuboid" scale="0.3,0.5,0.3" position="0.2,0,0.421" colour="grey30"/>
+  </object>
+  <object id="spring">
+    <use type="cylinder" scale="0.15,0.15,0.2" position="0,0,-0.1" colour="red"/>
+  </object>
+  <object id="spring-pair">
+    <use type="spring" position="0.0,-0.1,0.0"/>
+    <use type="spring" position="0.0,0.1,0.0"/>
   </object>
   <object id="axel">
-    <use type="wheel" position="-0.717,0,0"/>
-    <use type="wheel" position="0.717,0,0" rotation="0,3.14159,0"/>
+    <use type="wheel" position="-0.717,0,0" rotation="0,3.14159,0"/>
+    <use type="wheel" position="0.717,0,0"/>
+    <use type="cylinder" scale="0.1,0.1,2.2" position="-1.1,0,0.571" rotation="0,0,1.5708" colour="yellow"/>
+  </object>
+  <object id="bogie-side">
+    <use type="spring-pair" position="0,-1.1,0"/>
+    <use type="spring-pair" position="0,1.1,0"/>
+    <use type="cuboid" scale="0.2,4.3,0.15" position="0,0,-0.25" colour="darkgrey">
+      <face id="bottom" type="extrude" scale="1,0.3,1" position="0,0,-0.1"/>
+    </use>
+    <use type="cuboid" scale="0.2,4.3,0.15" position="0,0,0.1" colour="darkgrey">
+      <face id="top" type="extrude" scale="1,0.3,1" position="0,0,0.3"/>
+    </use>
   </object>
   <object id="bogie">
     <use type="axel" position="0,0,0"/>
     <use type="axel" position="0,2,0"/>
     <use type="axel" position="0,-2,0"/>
+    <use type="bogie-side" position="0.9,0,0.571"/>
+    <use type="bogie-side" position="-0.9,0,0.571"/>
+    <use type="cuboid" scale="1.8,0.6,0.5" position="0,1,0.4" colour="darkgrey"/>
+    <use type="cuboid" scale="1.8,0.6,0.5" position="0,-1,0.4" colour="darkgrey"/>
+    <use type="cuboid" scale="1.2,1.9,0.5" position="0,0,0.5" colour="grey30"/>
+    <use type="cuboid" scale="1,5.3,0.4" position="0,0,0.4" colour="darkgrey"/>
+    <use type="cuboid" scale="1.6,0.2,0.3" position="0,2.7,0.4" colour="grey30"/>
+  </object>
+  <object id="battery">
+    <use type="cuboid" scale="2.6,2.2,0.7" position="0,0,0.4" colour="slategrey">
+      <face id="bottom" type="extrude" scale="0.6,1,1" position="0,0,-0.3"/>
+    </use>
+  </object>
+  <object id="buffer">
+    <use type="cylinder" scale="0.3,0.3,0.3" colour="grey10"/>
+    <use type="cylinder" scale="0.6,0.6,0.1" position="0,0,0.3" colour="grey10"/>
+  </object>
+  <object id="buffers">
+    <use type="buffer" rotation="-1.5708,0,0" position="1.1,0,-0.3"/>
+    <use type="buffer" rotation="-1.5708,0,0" position="-1.1,0,-0.3"/>
+    <use type="cuboid" scale="2.65,0.1,0.6" position="0,-0.05,-0.6">
+      <face id="front" type="extrude" scale="0.9,1,0.4" position="0,-0.5,0.18">
+        <face id="front" type="extrude" scale="0.4,1,0" position="0,-0.5,0.09"/>
+      </face>
+    </use>
   </object>
   <asset p.typeid="RailVehicleClass" id="brush-47" name="Brush 47" length="19.38" wheelBase="15.70" maxSpeed="42.4688">
     <bodyMesh id="body" size="2.69,19.38,3.9">
-      <use type="cuboid" position="0,0,1.2" scale="2.69,19.38,1.5" colour="#2c4f5a">
+      <use type="cuboid" position="0,0,1.2" scale="2.69,19.38,0.3" colour="goldenrod">
         <face id="bottom" colour="#2c3539"/>
-        <face id="front" colour="goldenrod"/>
-        <face id="back" colour="goldenrod"/>
-        <face id="top" type="extrude" scale="1,0.95,1" position="0,0,0.8">
-          <face id="front" colour="#e1eff3"/>
-          <face id="back" colour="#e1eff3"/>
-          <face id="top" type="extrude" scale="0.5,0.8,0" position="0,0,0.4" smooth="true" colour="#aeb0b0"/>
+        <face id="top" type="extrude" position="0,0,1.05" colour="#2c4f5a">
+          <face id="front" colour="goldenrod"/>
+          <face id="back" colour="goldenrod"/>
+          <face id="top" type="extrude" scale="1,0.96,1" position="0,0,0.775">
+            <face id="front" colour="#e1eff3"/>
+            <face id="back" colour="#e1eff3"/>
+            <face id="top" type="extrude" scale="0.5,0.85,0" smooth="true" position="0,0,0.575" colour="#aeb0b0"/>
+          </face>
         </face>
       </use>
-      <use type="cuboid" position="0,0,0.2" scale="2.6,4.5,1" colour="grey30"/>
+      <use type="cuboid" position="0,0,0.3" scale="1.6,4,1" colour="darkslategrey"/>
+      <use type="battery" position="0,1.2,0"/>
+      <use type="battery" position="0,-1.2,0"/>
+      <use type="buffers" position="0,9.69,1.2" colour="grey20"/>
+      <use type="buffers" position="0,-9.69,1.2" colour="grey20" rotation="0,3.14159,0"/>
     </bodyMesh>
     <bogie id="bogie1">
-      <use type="bogie" position="0,-1.85,0" colour="#413b3a"/>
+      <use type="bogie" position="0,-1.85,0"/>
     </bogie>
     <bogie id="bogie2">
-      <use type="bogie" position="0,1.85,0" rotation="0,3.14159,0" colour="#413b3a"/>
+      <use type="bogie" position="0,1.85,0" rotation="0,3.14159,0"/>
     </bogie>
   </asset>
 </ilt>
diff --git a/test/test-assetFactory.cpp b/test/test-assetFactory.cpp
index ae5a88a..204ffb3 100644
--- a/test/test-assetFactory.cpp
+++ b/test/test-assetFactory.cpp
@@ -21,7 +21,7 @@ BOOST_GLOBAL_FIXTURE(ApplicationBase);
 BOOST_GLOBAL_FIXTURE(TestMainWindow);
 
 const std::filesystem::path TMP {"/tmp"};
-class FactoryFixture : public TestRenderOutputSize<glm::ivec2 {2048, 2048}>, public SceneProvider {
+class FactoryFixture : public TestRenderOutputSize<glm::ivec2 {2048, 1024}>, public SceneProvider {
 public:
 	FactoryFixture() : sceneRenderer {size, output} { }
 	~FactoryFixture()
@@ -45,8 +45,8 @@ public:
 	void
 	environment(const SceneShader &, const SceneRenderer & sceneRenderer) const override
 	{
-		sceneRenderer.setAmbientLight({.2, .2, .2});
-		sceneRenderer.setDirectionalLight({.3, .3, .3}, east + south + south + down, *this);
+		sceneRenderer.setAmbientLight({.4, .4, .4});
+		sceneRenderer.setDirectionalLight({.6, .6, .6}, east + south + south + down, *this);
 	}
 	void
 	shadows(const ShadowMapper & mapper) const override
@@ -57,7 +57,7 @@ public:
 	void
 	render(float dist = 10.f)
 	{
-		sceneRenderer.camera.setView({dist, dist, dist}, south + west + down);
+		sceneRenderer.camera.setView({-dist, dist * 1.2f, dist * 1.2f}, south + east + down);
 		sceneRenderer.render(*this);
 	}
 	Collection<const Renderable> objects;
@@ -71,7 +71,7 @@ BOOST_AUTO_TEST_CASE(brush47xml)
 {
 	auto mf = AssetFactory::loadXML(RESDIR "/brush47.xml");
 	BOOST_REQUIRE(mf);
-	BOOST_REQUIRE_EQUAL(6, mf->shapes.size());
+	BOOST_REQUIRE_GE(mf->shapes.size(), 6);
 	BOOST_CHECK(mf->shapes.at("plane"));
 	BOOST_CHECK(mf->shapes.at("cylinder"));
 	BOOST_CHECK(mf->shapes.at("cuboid"));
@@ -80,7 +80,7 @@ BOOST_AUTO_TEST_CASE(brush47xml)
 	auto bogie = mf->shapes.at("bogie");
 	BOOST_REQUIRE(bogie);
 	auto bogieObj = std::dynamic_pointer_cast<const Object>(bogie);
-	BOOST_CHECK_EQUAL(3, bogieObj->uses.size());
+	BOOST_CHECK_GE(bogieObj->uses.size(), 3);
 	BOOST_CHECK_EQUAL(1, mf->assets.size());
 	auto brush47 = mf->assets.at("brush-47");
 	BOOST_REQUIRE(brush47);
@@ -93,7 +93,7 @@ BOOST_AUTO_TEST_CASE(brush47xml)
 	auto railVehicle = std::make_shared<RailVehicle>(brush47rvc);
 	objects.objects.push_back(railVehicle);
 
-	render(20);
+	render();
 }
 BOOST_AUTO_TEST_SUITE_END();