Skip to content

Conversation

@Argmaster
Copy link
Collaborator

@Argmaster Argmaster commented Mar 20, 2025

As requested in discord thread by @ikabod-kee this pull request adds rotation mode that allows to accumulate (pile up) multiple blocks of same type as one block. Blocks can be piled up to 4 times every time hanging which texture is used by it.

As an example of use of pile rotation mode duckweed block was added with placeholder textures:

image

@ikabod-kee
Copy link
Collaborator

duckweed.zip
Here's a better texture you can use for this until Carrie makes one

@ikabod-kee
Copy link
Collaborator

I'll make a seperate PR to convert the autumn leaf piles to pile rotation

@Argmaster
Copy link
Collaborator Author

Argmaster commented Mar 20, 2025

I am not sure if we should do that.

I am also curious if maybe we could have a rotation mode that would cycle through different predefined models for different data values to make actually bigger piles of stuff or something like turtle eggs from minecraft.

@careeoki
Copy link
Contributor

I am not sure if we should do that.

I would like to do that.

I am also curious if maybe we could have a rotation mode that would cycle through different predefined models for different data values to make actually bigger piles of stuff or something like turtle eggs from minecraft.

That would be useful

@Argmaster
Copy link
Collaborator Author

With #1144 I think this should be customizable in block configuration and allow between 2 and 16 different states, as it may make sense to have more options for some blocks, hm?
@IntegratedQuantum @careeoki @ikabod-kee

@IntegratedQuantum
Copy link
Member

Yeah, it would be nice to have an optional number of pile states.

@ikabod-kee
Copy link
Collaborator

Agreed

@Argmaster
Copy link
Collaborator Author

How should the max number of pile states be stored? In blocks.zig like other block properties or perhaps should it be stored by rotation code?

@IntegratedQuantum
Copy link
Member

I would prefer if it was stored in the particular rotation mode, since most of the blocks have no use for this property and there are too many random block properties already in my opinion.

@careeoki
Copy link
Contributor

I have a pull request with new duckweed textures and a unique item texture.

@Argmaster
Copy link
Collaborator Author

State count is now configurable. Rotation mode was renamed to texturePile to leave space for modelPile in the future, unless objected.

@Argmaster
Copy link
Collaborator Author

I have a pull request with new duckweed textures and a unique item texture.

Do you want to merge it into this PR or wait until this PR is merged?

@careeoki
Copy link
Contributor

Do you want to merge it into this PR or wait until this PR is merged?

I already opened a PR to your fork, so you can merge it.

@IntegratedQuantum IntegratedQuantum merged commit 07f5d82 into PixelGuys:master Mar 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants