Fix model jankiness when flying as a pony (only rotate when moving)

This commit is contained in:
Sollace 2022-10-08 15:44:44 +02:00
parent 5680e60464
commit 82f6ce746d
2 changed files with 47 additions and 21 deletions

View file

@ -21,6 +21,7 @@ import net.minecraft.entity.Entity;
import net.minecraft.entity.player.PlayerEntity;
import net.minecraft.util.Identifier;
import net.minecraft.util.math.MathHelper;
import net.minecraft.util.math.Vec3d;
public class Main extends MineLPDelegate implements ClientModInitializer {
@ -48,7 +49,10 @@ public class Main extends MineLPDelegate implements ClientModInitializer {
if (pony.getMotion().isFlying()) {
model.getAttributes().wingAngle = MathHelper.clamp(pony.getMotion().getWingAngle() / 3 - (float)Math.PI * 0.7F, -3, 0);
model.getAttributes().isHorizontal = true;
Vec3d motion = entity.getVelocity();
double zMotion = Math.sqrt(motion.x * motion.x + motion.z * motion.z);
model.getAttributes().isGoingFast |= zMotion > 0.4F;
}
model.getAttributes().isGoingFast |= pony.getMotion().isRainbooming();

View file

@ -5,6 +5,7 @@ import com.minelittlepony.unicopia.ability.magic.SpellPredicate;
import com.minelittlepony.unicopia.ability.magic.spell.effect.SpellType;
import com.minelittlepony.unicopia.advancement.UCriteria;
import com.minelittlepony.unicopia.block.data.ModificationType;
import com.minelittlepony.unicopia.client.minelittlepony.MineLPDelegate;
import com.minelittlepony.unicopia.client.render.PlayerPoser.Animation;
import com.minelittlepony.unicopia.entity.*;
import com.minelittlepony.unicopia.entity.duck.LivingEntityDuck;
@ -44,6 +45,7 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
private float thrustScale = 0;
private boolean flapping;
private boolean isCancelled;
private int prevStrafe;
private float strafe;
@ -137,7 +139,10 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
.orElse(pony.getSpecies().getFlightType());
}
private void cancelFlight() {
public void cancelFlight(boolean force) {
if (force) {
isCancelled = true;
}
boolean wasFlying = isFlyingEither;
entity.getAbilities().flying = false;
isFlyingEither = false;
@ -148,6 +153,9 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
if (wasFlying) {
entity.calculateDimensions();
}
pony.setDirty();
entity.sendAbilitiesUpdate();
}
private double getHorizontalMotion() {
@ -195,8 +203,12 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
boolean startedFlyingCreative = !creative && isFlyingEither != entity.getAbilities().flying;
if (!creative) {
if (entity.world.isClient && entity.isOnGround()) {
cancelFlight();
if (entity.isOnGround() || isCancelled) {
cancelFlight(false);
}
if (entity.isOnGround()) {
isCancelled = false;
}
entity.getAbilities().flying |= (type.canFly() || entity.getAbilities().allowFlying) && isFlyingEither;
@ -214,7 +226,7 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
return;
}
cancelFlight();
cancelFlight(false);
}
@ -222,7 +234,7 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
if (isGravityNegative()) {
if (entity.isOnGround() || (!creative && entity.horizontalCollision)) {
cancelFlight();
cancelFlight(false);
}
if (entity.isClimbing() && (entity.horizontalCollision || ((LivingEntityDuck)entity).isJumping())) {
@ -287,8 +299,8 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
entity.setVelocity(velocity.toImmutable());
if (isFlying() && !entity.isFallFlying() && !pony.isHanging()) {
if (isFlying() && !entity.isFallFlying() && !pony.isHanging() && pony.isClient()) {
if (MineLPDelegate.getInstance().getPlayerPonyRace(entity).isDefault() && getHorizontalMotion() > 0.03) {
float pitch = ((LivingEntityDuck)entity).getLeaningPitch();
if (pitch < 1) {
if (pitch < 0.9F) {
@ -297,6 +309,7 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
pitch += 0.09F;
((LivingEntityDuck)entity).setLeaningPitch(Math.max(0, pitch));
}
}
entity.limbAngle = 20 + (float)Math.cos(entity.age / 7F) - 0.5F;
entity.limbDistance = thrustScale;
@ -380,7 +393,7 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
if (!getFlightType().canFly()) {
entity.playSound(USounds.ITEM_ICARUS_WINGS_EXHAUSTED, 1, 2);
cancelFlight();
cancelFlight(false);
}
}
}
@ -422,11 +435,8 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
&& (horMotion > 0.2 || (motion > 0.2 && velocity.y < -0.02 * getGravitySignum()));
boolean fallingTakeOffCondition = !entity.isOnGround() && velocity.y < -1.6 * getGravitySignum();
if ((takeOffCondition || fallingTakeOffCondition) && !pony.isHanging()) {
entity.getAbilities().flying = true;
isFlyingEither = true;
isFlyingSurvival = true;
entity.calculateDimensions();
if ((takeOffCondition || fallingTakeOffCondition) && !pony.isHanging() && !isCancelled) {
startFlying(false);
if (!isGravityNegative()) {
velocity.y += horMotion + 0.3;
@ -438,6 +448,16 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
}
}
public void startFlying(boolean force) {
if (force) {
isCancelled = false;
}
entity.getAbilities().flying = true;
isFlyingEither = true;
isFlyingSurvival = true;
entity.calculateDimensions();
}
private void handleWallCollission(MutableVector velocity) {
if (wallHitCooldown > 0) {
return;
@ -470,7 +490,7 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
}
entity.setVelocity(velocity.toImmutable());
cancelFlight();
cancelFlight(false);
}
private void moveFlying(MutableVector velocity) {
@ -614,6 +634,7 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
public void toNBT(NbtCompound compound) {
super.toNBT(compound);
compound.putBoolean("isFlying", isFlyingSurvival);
compound.putBoolean("isCancelled", isCancelled);
compound.putBoolean("isFlyingEither", isFlyingEither);
compound.putInt("ticksInAir", ticksInAir);
}
@ -622,6 +643,7 @@ public class PlayerPhysics extends EntityPhysics<PlayerEntity> implements Tickab
public void fromNBT(NbtCompound compound) {
super.fromNBT(compound);
isFlyingSurvival = compound.getBoolean("isFlying");
isCancelled = compound.getBoolean("isCancelled");
isFlyingEither = compound.getBoolean("isFlyingEither");
ticksInAir = compound.getInt("ticksInAir");