- Added InheritsAngularVel to Gibs and Emissions

- Added InheritsVelWhenDetached and InheritsAngularVelWhenDetached to Attachables
- Exposed these to Lua

Author: comradeshook

CHANGELOG.md

@@ -52,7 +52,11 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 - Allow lua scripts to use LuaJIT's BitOp module (see https://bitop.luajit.org/api.html)
 
-- New `Emission` INI and Lua (R/W) property `ParticleCount` which sets how many particles the Emission spawns per emission.
+- New `Emission` INI and Lua (R/W) property `ParticleCount` which sets how many particles the Emission spawns per emission. Defaults to 1.
+
+- New `Gib` and `Emission` INI and Lua (R/W) property `InheritsAngularVel`, which determines how much of the parent MO's angular velocity they inherit. Defaults to 1 for gibs, 0 for emissions.
+
+- New `Attachable` INI and Lua (R/W) properties `InheritsVelWhenDetached` and `InheritsAngularVelWhenDetached`, which determine how much of these velocities an attachable inherits from its parent when detached. Defaults to 1.
 
 </details>
 
@@ -85,9 +89,11 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 - Almost all ctrl+* special inputs functionality (i.e restarting activity, world dumps, showing performance stats) are now mapped to right alt, to not interfere with default crouching inputs. The only exception is ctrl+arrow keys for changing console size.
 
-- Gibs and detached Attachables now inherit the parent's angular velocity, as well as velocity derived from the angular velocity of the parent MO and their offset from the parent's centre. On gibs, this is scaled by `InheritsVel`.
+- `Gib`s and detached `Attachable`s now inherit the parent's angular velocity by default.
+
+- `InheritsVel` now accounts for the angular velocity of the parent MO, resulting in offset gibs and emissions being flung further away.
 
-- `Emissions` with `InheritsVel` now also inherit velocity and angular velocity derived from the angular velocity of the parent.
+- `InheritsVel` and its ilk have been uncapped, allowing users to set them outside of 0-1.
 
 </details>
 

Source/Entities/Attachable.cpp

@@ -47,8 +47,8 @@ void Attachable::Clear() {
 	m_InheritsRotAngle = true;
 	m_InheritedRotAngleOffset = 0;
 	m_InheritsFrame = false;
-	m_InheritsVelWhenDetached = 0.0F;
-	m_InheritsAngularVelWhenDetached = 0.0F;
+	m_InheritsVelWhenDetached = 1.0F;
+	m_InheritsAngularVelWhenDetached = 1.0F;
 
 	m_AtomSubgroupID = -1L;
 	m_CollidesWithTerrainWhileAttached = true;

Source/Entities/Attachable.h

@@ -271,11 +271,20 @@ namespace RTE {
 
 		/// How much of the root parent's velocity this attachable inherits when detached
 		/// @return The proportion of the velocity inherited. 0.1 = 10% inheritance.
-		float InheritsVelocityWhenDetached() { return m_InheritsVelWhenDetached; }
+		float InheritsVelocityWhenDetached() const { return m_InheritsVelWhenDetached; }
 
 		/// How much of the root parent's angular velocity this attachable inherits when detached
 		/// @return The proportion of the angular velocity inherited. 0.1 = 10% inheritance.
-		float InheritsAngularVelocityWhenDetached() { return m_InheritsAngularVelWhenDetached; }
+		float InheritsAngularVelocityWhenDetached() const { return m_InheritsAngularVelWhenDetached; }
+
+		/// Sets how much of the root parent's velocity this attachable inherits when detached
+		/// @param The proportion of the velocity inherited. 0.1 = 10% inheritance.
+		void SetInheritsVelocityWhenDetached(float newValue) { m_InheritsVelWhenDetached = newValue; }
+
+		/// Sets how much of the root parent's angular velocity this attachable inherits when detached
+		/// @param The proportion of the angular velocity inherited. 0.1 = 10% inheritance.
+		void SetInheritsAngularVelocityWhenDetached(float newValue) { m_InheritsAngularVelWhenDetached = newValue; }
+
 #pragma endregion
 
 #pragma region Collision Management

Source/Entities/Emission.h

@@ -114,13 +114,21 @@ namespace RTE {
 			m_StopTimer.Reset();
 		}
 
-		/// How much of the root parent's velocity this emission inherit
+		/// How much of the root parent's velocity this emission inherits
 		/// @return The proportion of the velocity inherited. 0.1 = 10% inheritance.
-		float InheritsVelocity() { return m_InheritsVel; }
+		float InheritsVelocity() const { return m_InheritsVel; }
 
-		/// How much of the root parent's angular velocity this emission inherit
+		/// How much of the root parent's angular velocity this emission inherits
 		/// @return The proportion of the angular velocity inherited. 0.1 = 10% inheritance.
-		float InheritsAngularVelocity() { return m_InheritsAngularVel; }
+		float InheritsAngularVelocity() const { return m_InheritsAngularVel; }
+
+		/// Sets how much of the root parent's velocity this emission inherits
+		/// @param The proportion of the velocity inherited. 0.1 = 10% inheritance.
+		void SetInheritsVelocity(float newValue) { m_InheritsVel = newValue; }
+
+		/// Sets how much of the root parent's angular velocity this emission inherits
+		/// @param The proportion of the angular velocity inherited. 0.1 = 10% inheritance.
+		void SetInheritsAngularVelocity(float newValue) { m_InheritsAngularVel = newValue; }
 
 		/// Gets offset of the emission point from Emitter's sprite center, which gets rotated with owner Emitter
 		/// @return Returns emission offset.

Source/Entities/Gib.cpp

@@ -23,6 +23,7 @@ void Gib::Clear() {
 	m_MaxVelocity = 0;
 	m_LifeVariation = 0.1F;
 	m_InheritsVel = 1.0F;
+	m_InheritsAngularVel = 1.0F;
 	m_IgnoresTeamHits = false;
 	m_SpreadMode = SpreadMode::SpreadRandom;
 }
@@ -36,6 +37,7 @@ int Gib::Create(const Gib& reference) {
 	m_MaxVelocity = reference.m_MaxVelocity;
 	m_LifeVariation = reference.m_LifeVariation;
 	m_InheritsVel = reference.m_InheritsVel;
+	m_InheritsAngularVel = reference.m_InheritsAngularVel;
 	m_IgnoresTeamHits = reference.m_IgnoresTeamHits;
 	m_SpreadMode = reference.m_SpreadMode;
 
@@ -56,6 +58,7 @@ int Gib::ReadProperty(const std::string_view& propName, Reader& reader) {
 	MatchProperty("MaxVelocity", { reader >> m_MaxVelocity; });
 	MatchProperty("LifeVariation", { reader >> m_LifeVariation; });
 	MatchProperty("InheritsVel", { reader >> m_InheritsVel; });
+	MatchProperty("InheritsAngularVel", { reader >> m_InheritsAngularVel; });
 	MatchProperty("IgnoresTeamHits", { reader >> m_IgnoresTeamHits; });
 	MatchProperty("SpreadMode", { m_SpreadMode = static_cast<SpreadMode>(std::stoi(reader.ReadPropValue())); });
 
@@ -89,6 +92,8 @@ int Gib::Save(Writer& writer) const {
 	writer << m_LifeVariation;
 	writer.NewProperty("InheritsVel");
 	writer << m_InheritsVel;
+	writer.NewProperty("InheritsAngularVel");
+	writer << m_InheritsAngularVel;
 	*/
 
 	return 0;

Source/Entities/Gib.h

@@ -86,6 +86,10 @@ namespace RTE {
 		/// @return The proportion of inherited velocity as a scalar from 0 to 1.
 		float InheritsVelocity() const { return m_InheritsVel; }
 
+		/// Gets how much of the gibbing parent's angular velocity this Gib's GibParticles should inherit.
+		/// @return The proportion of inherited velocity as a scalar from 0 to 1.
+		float InheritsAngularVelocity() const { return m_InheritsAngularVel; }
+
 		/// Gets whether this Gib's GibParticles should ignore hits with the team of the gibbing parent.
 		/// @return Whether this Gib's GibParticles should ignore hits with the team of the gibbing parent.
 		bool IgnoresTeamHits() const { return m_IgnoresTeamHits; }
@@ -108,6 +112,7 @@ namespace RTE {
 		float m_MaxVelocity; //!< The maximum velocity a GibParticle object can have when spawned.
 		float m_LifeVariation; //!< The per-Gib variation in Lifetime, in percentage of the existing Lifetime of the gib.
 		float m_InheritsVel; //!< How much of the exploding parent's velocity this Gib should inherit.
+		float m_InheritsAngularVel; //!< How much of the exploding parent's angular velocity this Gib should inherit.
 		bool m_IgnoresTeamHits; //!< Whether this Gib should ignore hits with the team of the exploding parent or not.
 		SpreadMode m_SpreadMode; //!< Determines what kind of logic is used when applying velocity to the GibParticle objects.
 

Source/Entities/MOSRotating.cpp

@@ -943,12 +943,10 @@ void MOSRotating::CreateGibsWhenGibbing(const Vector& impactImpulse, MovableObje
 				Vector gibVelocity(radius * scale + minVelocity, 0);
 				gibVelocity.RadRotate(randAngle + RandomNum(0.0F, spread) + static_cast<float>(i) * goldenAngle);
 
-				if (gibSettingsObject.InheritsVelocity() > 0) {
-					Vector offsetFromRootParent = m_Pos - GetRootParent()->GetPos() + rotatedGibOffset;
-					Vector rotationalVelocity = (offsetFromRootParent.GetPerpendicular() * GetRootParent()->GetAngularVel() * gibSettingsObject.InheritsVelocity()) / c_PPM;
-					gibVelocity += rotationalVelocity;
-					gibParticleClone->SetAngularVel(gibParticleClone->GetAngularVel() + GetRootParent()->GetAngularVel() * gibSettingsObject.InheritsVelocity());
-				}
+				Vector offsetFromRootParent = m_Pos - GetRootParent()->GetPos() + rotatedGibOffset;
+				Vector rotationalVelocity = (offsetFromRootParent.GetPerpendicular() * GetRootParent()->GetAngularVel() * gibSettingsObject.InheritsVelocity()) / c_PPM;
+				gibVelocity += rotationalVelocity;
+				gibParticleClone->SetAngularVel(gibParticleClone->GetAngularVel() + GetRootParent()->GetAngularVel() * gibSettingsObject.InheritsAngularVelocity());
 
 				if (lifetime != 0) {
 					gibParticleClone->SetLifetime(std::max(static_cast<int>(static_cast<float>(lifetime) * (1.0F - lifeVariation * ((radius / maxRadius) * 0.75F + RandomNormalNum() * 0.25F))), 1));
@@ -1007,12 +1005,10 @@ void MOSRotating::CreateGibsWhenGibbing(const Vector& impactImpulse, MovableObje
 					gibVelocity.RadRotate(gibSpread * RandomNormalNum());
 				}
 
-				if (gibSettingsObject.InheritsVelocity() > 0) {
-					Vector offsetFromRootParent = m_Pos - GetRootParent()->GetPos() + rotatedGibOffset;
-					Vector rotationalVelocity = (offsetFromRootParent.GetPerpendicular() * GetRootParent()->GetAngularVel() * gibSettingsObject.InheritsVelocity()) / c_PPM;
-					gibVelocity += rotationalVelocity;
-					gibParticleClone->SetAngularVel(gibParticleClone->GetAngularVel() + GetRootParent()->GetAngularVel() * gibSettingsObject.InheritsVelocity());
-				}
+				Vector offsetFromRootParent = m_Pos - GetRootParent()->GetPos() + rotatedGibOffset;
+				Vector rotationalVelocity = (offsetFromRootParent.GetPerpendicular() * GetRootParent()->GetAngularVel() * gibSettingsObject.InheritsVelocity()) / c_PPM;
+				gibVelocity += rotationalVelocity;
+				gibParticleClone->SetAngularVel(gibParticleClone->GetAngularVel() + GetRootParent()->GetAngularVel() * gibSettingsObject.InheritsAngularVelocity());
 
 				gibParticleClone->SetVel(gibVelocity + ((m_PrevVel + m_Vel) / 2) * gibSettingsObject.InheritsVelocity());
 
@@ -1478,9 +1474,9 @@ Attachable* MOSRotating::RemoveAttachable(Attachable* attachable, bool addToMova
 	}
 	RTEAssert(attachable->IsAttachedTo(this), "Tried to remove Attachable " + attachable->GetPresetNameAndUniqueID() + " from presumed parent " + GetPresetNameAndUniqueID() + ", but it had a different parent (" + (attachable->GetParent() ? attachable->GetParent()->GetPresetNameAndUniqueID() : "ERROR") + "). This should never happen!");
 
-	Vector rotationalVelocity = ((attachable->GetPos() - GetRootParent()->GetPos()).GetPerpendicular() * GetRootParent()->GetAngularVel()) / c_PPM;
-	attachable->SetAngularVel(attachable->GetAngularVel() + GetRootParent()->GetAngularVel());
-	attachable->SetVel(attachable->GetVel() + rotationalVelocity); // Attachables have already had their velocity updated by ApplyImpulses(), no need to add impactImpulse again
+	Vector rotationalVelocity = ((attachable->GetPos() - GetRootParent()->GetPos()).GetPerpendicular() * GetRootParent()->GetAngularVel() * attachable->InheritsVelocityWhenDetached()) / c_PPM;
+	attachable->SetAngularVel(attachable->GetAngularVel() + GetRootParent()->GetAngularVel() * attachable->InheritsAngularVelocityWhenDetached());
+	attachable->SetVel(attachable->GetVel() * attachable->InheritsVelocityWhenDetached() + rotationalVelocity); // Attachables have already had their velocity updated by ApplyImpulses(), no need to add impactImpulse again
 
 	if (!m_Attachables.empty()) {
 		m_Attachables.remove(attachable);

Source/Lua/LuaBindingsEntities.cpp

@@ -549,6 +549,8 @@ LuaBindingRegisterFunctionDefinitionForType(EntityLuaBindings, Attachable) {
 	    .property("CanCollideWithTerrain", &Attachable::CanCollideWithTerrain)
 	    .property("DrawnAfterParent", &Attachable::IsDrawnAfterParent, &Attachable::SetDrawnAfterParent)
 	    .property("InheritsFrame", &Attachable::InheritsFrame, &Attachable::SetInheritsFrame)
+	    .property("InheritsVelWhenDetached", &Attachable::InheritsVelocityWhenDetached, &Attachable::SetInheritsVelocityWhenDetached)
+	    .property("InheritsAngularVelWhenDetached", &Attachable::InheritsAngularVelocityWhenDetached, &Attachable::SetInheritsAngularVelocityWhenDetached)
 
 	    .def("IsAttached", &Attachable::IsAttached)
 	    .def("IsAttachedTo", &Attachable::IsAttachedTo)
@@ -581,6 +583,8 @@ LuaBindingRegisterFunctionDefinitionForType(EntityLuaBindings, Emission) {
 	    .property("Spread", &Emission::GetSpread, &Emission::SetSpread)
 	    .property("Offset", &Emission::GetOffset, &Emission::SetOffset)
 	    .property("ParticleCount", &Emission::GetParticleCount, &Emission::SetParticleCount)
+	    .property("InheritsVel", &Emission::InheritsVelocity, &Emission::SetInheritsVelocity)
+	    .property("InheritsAngularVel", &Emission::InheritsAngularVelocity, &Emission::SetInheritsAngularVelocity)
 
 	    .def("ResetEmissionTimers", &Emission::ResetEmissionTimers);
 }
@@ -598,6 +602,7 @@ LuaBindingRegisterFunctionDefinitionForType(EntityLuaBindings, Gib) {
 	    .def_readwrite("Spread", &Gib::m_Spread)
 	    .def_readwrite("LifeVariation", &Gib::m_LifeVariation)
 	    .def_readwrite("InheritsVel", &Gib::m_InheritsVel)
+	    .def_readwrite("InheritsAngularVel", &Gib::m_InheritsAngularVel)
 	    .def_readwrite("IgnoresTeamHits", &Gib::m_IgnoresTeamHits)
 
 	    .enum_("SpreadMode")[luabind::value("SpreadRandom", Gib::SpreadMode::SpreadRandom),