[1.14.2] TileEntity rendering + data syncing (+ TileEntity tick function every other tick)

[Simon_kungen]

Hi

 

I got my TileEntity working good now, now I want to add a type of render to it, and looking through the wiki the optimal way of doing it would be to use a TileEntityRendererFast. I would not say my TileEntity would be hard to render as it would only have to render a cuboid with varying alpha and texture depending on the TileEntity variables.

The issue I'm facing is that I have no idea how to do that, and better yet how to send the data to the client efficiently. I have it do stuff every tick, which is only changing one variable out of the three variables it stores, so I'm pretty sure using a custom message protocol is needed. The other two is what texture to use and whenever it should do a particle effect at the block (same texture). But I'm a bit lost about all this, going to write a summary and what little code I have on the TileEntityRendererFast and the actul TileEntity below:

 

A Tree Tap you place on a tree, I got the bucket and actual Tree Tap model working fine. The TileEntity stores what type of liquid it is (resin, water or none), whenever it can make the "bucket" start filling up with a particle effect and what volume there is in the bucket.

Something I was wondering is if there is a way to have a if statement that would only pass like every other tick or less, to simulate viscosity of different fluids and have the "drip" animation only happen that tick or something. That would also free up some of the network if I only need to send new information that particular tick.

 

 

 

TreeTapTileEntityRenderer.java

Spoiler

public class TreeTapTileEntityRenderer extends TileEntityRendererFast<TreeTapTileEntity>
{


    @Override
    public void renderTileEntityFast(TreeTapTileEntity te, double x, double y, double z, float partialTicks, int destroyStage, BufferBuilder buffer)
    {


    }
}

 

 

 

TreeTapTileEntity.java

Spoiler

public class TreeTapTileEntity extends TileEntity implements ITickableTileEntity
{

    private int volume = 0;
    private boolean canFill = false;
    private FluidType fluidType = FluidType.NONE;

    public TreeTapTileEntity()
    {
        super(IntercraftTileEntities.TREETAP);
    }


    @Override
    public void tick()
    {
        if (canFill) {
            if (volume < 1000) {
                setVolume(volume+1);

                


            }
        }
    }


    @Override
    public CompoundNBT write(CompoundNBT compound)
    {
        compound.putInt("volume", volume);
        compound.putBoolean("can_fill",canFill);
        compound.putString("fluid_type",fluidType.toString());

        return super.write(compound);
    }

    @Override
    public void read(CompoundNBT compound)
    {
        super.read(compound);
        volume = compound.getInt("volume");
        canFill = compound.getBoolean("can_fill");
        fluidType = FluidType.valueOf(compound.getString("fluid_type").toUpperCase());

    }


    public int getVolume()
    {
        return volume;
    }

    public boolean getCanFill()
    {
        return canFill;
    }

    public FluidType getFluidType()
    {
        return this.fluidType;
    }

    public void setVolume(int value)
    {
        volume = value;
        markDirty();
    }

    public void setCanFill(boolean bool)
    {
        canFill = bool;
        markDirty();
    }

    public void setFluidType(FluidType type)
    {
        fluidType = type;
        markDirty();
    }
}

 

 

 

[DavidM]

7 hours ago, Simon_kungen said:

if there is a way to have a if statement that would only pass like every other tick

World#getTotalWorldTime (or something similar; can’t remember the spelling) returns the current world time. Modulo the total world time by 2 (or more if you would like the delay between triggers to be longer) and check if the result is equal to 0.

 

Use packets to send data from server to client.

[Simon_kungen]

11 hours ago, DavidM said:

World#getTotalWorldTime (or something similar; can’t remember the spelling) returns the current world time. Modulo the total world time by 2 (or more if you would like the delay between triggers to be longer) and check if the result is equal to 0.

 

Use packets to send data from server to client.

Alright, that worked pretty well. I added this to my TileEntity;

Spoiler

@Override
public CompoundNBT getUpdateTag()
{
    CompoundNBT compound = super.getUpdateTag();
    compound.putInt("volume", volume);
    compound.putBoolean("can_fill",canFill);
    compound.putString("fluid_type",fluidType.getName());
    return compound;
}

@Override
public void handleUpdateTag(CompoundNBT compound)
{
    volume = compound.getInt("volume");
    canFill = compound.getBoolean("can_fill");
    fluidType = FluidType.valueOf(compound.getString("fluid_type").toUpperCase());
    super.handleUpdateTag(compound);
}

 

 

but when I tried to add a block update mechanic to it it wouldn't update:

Spoiler

@Override
public void onDataPacket(NetworkManager net, SUpdateTileEntityPacket pkt)
{
    CompoundNBT compound = pkt.getNbtCompound();
    volume = compound.getInt("volume");
    canFill = compound.getBoolean("can_fill");
    fluidType = FluidType.valueOf(compound.getString("fluid_type").toUpperCase());
    world.notifyBlockUpdate(pos,world.getBlockState(pos),world.getBlockState(pos),2);
}

@Nullable
@Override
public SUpdateTileEntityPacket getUpdatePacket()
{
    CompoundNBT compound = super.getUpdateTag();
    compound.putInt("volume", volume);
    compound.putBoolean("can_fill",canFill);
    compound.putString("fluid_type",fluidType.getName());

    return new SUpdateTileEntityPacket(pos, 1, compound);
}

 

 

 

I would like to add my own packet message to only send the data needed for that particular moment, but for now, a quick and dirty method would probably suffice for the time being.

 

Something I do wonder more is how I'm supposed to do the TileEntityRendererFast.

I registered this to the client, but the problem is that I have no idea how to work with this.

@Override
public void renderTileEntityFast(TreeTapTileEntity te, double x, double y, double z, float partialTicks, int destroyStage, BufferBuilder buffer)
{
    // *Confusion sound*

    //buffer.begin(GL11.GL_QUADS, DefaultVertexFormats.BLOCK);



}

 

A 3D cuboid with varying texture and height and of course what axis it should align to (in my blockstate). I tried to search for some example online how to work with this, but none of the uses was sufficient to me.

https://github.com/search?l=&q=TileEntityRendererFast+language%3AJava&type=Code

[Simon_kungen]

If anyone could tell me what I'm doing wrong, I would gladly accept any answer:

@Override
public void renderTileEntityFast(TreeTapTileEntity te, double x, double y, double z, float partialTicks, int destroyStage, BufferBuilder buffer)
{
    // *Confusion sound*


    BlockPos pos = te.getPos();

    net.minecraft.world.IEnviromentBlockReader world = MinecraftForgeClient.getRegionRenderCache(te.getWorld(), pos);
    BlockState state = world.getBlockState(pos);


    if (state.get(BlockProperties.BUCKET) != BucketType.NONE) {
        buffer.setTranslation(x - pos.getX(), y - pos.getY(), z - pos.getZ());


        buffer.pos(1,1,1).color(1,1,1,1);


    }


}

 

Here I don't really expect much except for absolutely anything to happen.

[Simon_kungen]

Alright, I got some help getting something to render, but as they said you have to keep in mind what direction it is supposed to render it from, which I have no idea how to do.

buffer.pos(0,0,0).color(0,0,1,1).endVertex();
buffer.pos(4,0,0).color(0,0,1,1).endVertex();
buffer.pos(0,0,4).color(0,0,1,1).endVertex();
buffer.pos(4,0,4).color(0,0,1,1).endVertex();

 

Something I realized was that the fluid level is only going to be seen from the top, which would mean I only need to render a single side (top) and then change the y value to represent the fluid level.