Projectile rendering incorrectly when given a high velocity

[MysticalMike 0]

When I spawn a projectile with a high velocity it seems to render in the incorrect location but it ends up hitting the correct block. Better explained by watching this clip: https://youtu.be/ENmASw9UQVQ

 

Here is the basic code I used to spawn the projectile:

 

The only thing that I could think of as the cause would be some kind of de-sync with the client because the projectile doesn't hit mobs or blocks that would be in the way if it was actually where it's rendering.

Any clues on how to fix this would be very helpful.

[diesieben07 7706]

ArrowEntity uses SSpawnObjectPacket, which can only handle motion between -3.9 and +3.9.

[MysticalMike 0]

On 2/19/2021 at 1:19 AM, diesieben07 said:

ArrowEntity uses SSpawnObjectPacket, which can only handle motion between -3.9 and +3.9.

Okay I see what you mean in SSpawnObjectPacket but I'm not sure how I could change that or if there's a different packet I could send. I tried creating a new packet extending SSpawnObjectPacket but minecraft doesn't render custom non-living entities with that packet. Before I was using 'NetworkHooks.getEntitySpawningPacket(this);' which ultimately creates a SCustomPayloadPlayPacket packet which does render custom non-living entities but I don't know how I could edit that packet either. (Both clamp the motion)

 

Edited February 20 by MysticalMike

[diesieben07 7706]

Do not make your own "vanilla style" packet (i.e. subclass of IPacket). It won't work.

 

You can implement IEntityAdditionalSpawnData on your entity and encode the motion again, without clamping. This requires the use of NetworkHooks.getEntitySpawningPacket (like all modded entities).

[MysticalMike 0]

On 2/20/2021 at 12:03 PM, diesieben07 said:

Do not make your own "vanilla style" packet (i.e. subclass of IPacket). It won't work.

 

You can implement IEntityAdditionalSpawnData on your entity and encode the motion again, without clamping. This requires the use of NetworkHooks.getEntitySpawningPacket (like all modded entities).

Okay so I got it to write and read my data but how can I use that to actually change the speed for the client?

[diesieben07 7706]

Is the method called? Does it read the correct values?

[MysticalMike 0]

2 minutes ago, diesieben07 said:

Is the method called? Does it read the correct values?

Yeah, but even setting the motion even to (0, 0, 0) doesn't seem to change it.

[diesieben07 7706]

I can't see why that would happen.

Can you post a Git repo of your mod?

[MysticalMike 0]

3 hours ago, diesieben07 said:

I can't see why that would happen.

Can you post a Git repo of your mod?

Okay I posted it to my github: https://github.com/MysticalMike/GunMod

I use EMPGun to spawn the projectile: io/github/mysticalmike/gunmod/common/items/EMPGun

And BulletEntity is the projectile spawned: io/github/mysticalmike/gunmod/common/entities/BulletEntity

Edited February 22 by MysticalMike

[MysticalMike 0]

I'm just going to bump this because I don't think you got a notification since I edited the last message, but there's my project ^^^

[diesieben07 7706]

Yes... that's why you don't edit posts if you have significant new info... Anyways.

 

SEntityVelocityPacket has the same issue. So you have to disable the default velocity updates (EntityType.Builder#setShouldReceiveVelocityUpdates) and send them yourself using a custom packet.

[MysticalMike 0]

What custom packet do I send?

[diesieben07 7706]

You need to write it.

[MysticalMike 0]

On 2/20/2021 at 12:03 PM, diesieben07 said:

Do not make your own "vanilla style" packet (i.e. subclass of IPacket). It won't work.

 

You can implement IEntityAdditionalSpawnData on your entity and encode the motion again, without clamping. This requires the use of NetworkHooks.getEntitySpawningPacket (like all modded entities).

How exactly does one write a packet and get forge to send it?

[diesieben07 7706]

https://mcforge.readthedocs.io/en/latest/networking/simpleimpl/

 

As for how to send it, that's on you. You have to send it.