Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix MC-248964 #95

Open
muzikbike opened this issue Sep 12, 2022 · 5 comments
Open

Fix MC-248964 #95

muzikbike opened this issue Sep 12, 2022 · 5 comments
Labels
suggestion A suggestion, enhancement, feature or request

Comments

@muzikbike
Copy link

The bug (https://bugs.mojang.com/browse/MC-248964) concerns sugar cane with custom models not culling against each other, although other non-solid plant blocks like tall grass would also likely be affected if given such models.

The resource pack for testing this can be found attached to the ticket.
2022-09-12_23 44 24

@FxMorin
Copy link
Owner

FxMorin commented Dec 15, 2022

Have you tested and checked that it does not work using MoreCulling, if so can you include the resource pack

@FxMorin FxMorin added Awaiting Response Waiting for a response suggestion A suggestion, enhancement, feature or request and removed Awaiting Response Waiting for a response labels Dec 15, 2022
@FxMorin
Copy link
Owner

FxMorin commented Dec 15, 2022

I just remembered that my culling technique is written so that blocks without collisions don't get affected.
I think the system has progressed enough that I can remove that condition. I'll check that when I have some time

@FxMorin
Copy link
Owner

FxMorin commented Dec 19, 2022

After testing ever single non colliding block in the game. Seems my system no longer needs that check, with all the code I've written its all taken care of. So this will probably increase performance for some blocks and also fix this bug and all other bugs related to non colliding blocks like rails.
I'm probably going to publish this in the beginning of January tho since I wont be home till than

@FxMorin
Copy link
Owner

FxMorin commented Jan 6, 2023

Alright so we are one step closer to fixing this bug. In v0.12.5 I made it possible for none colliding blocks to cull.
Had a couple bugs in it that got fixed in v0.13.0, although this still does not work.
Only difference between the two are the AbstractBlockSettings: collidable & opaque
This makes sense since opaque basically bypasses all my extra checks, so I have two options:

  1. Add a new tag within resourcepacks that people can use to declare a model as opaque internally so I can skip a whole load of work.
  2. Run complicated equations to determine the model's size, translucency, and transformations and than attempt to do a complicated match against the other sugar cane.

So ill probably just add a new tag

Edit:
I may actually try seeing if its possible to override the cull shape using a generated one based on the model. This would only be created when using a custom model, it would fix a couple of bugs although its also pretty expensive

@FxMorin
Copy link
Owner

FxMorin commented Jan 7, 2024

Cull shapes where added to MoreCulling, Anyone want to try and see if this is still an issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
suggestion A suggestion, enhancement, feature or request
Projects
None yet
Development

No branches or pull requests

2 participants