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

[Enhancement] Split tracingDistance to EntityTracingDistance and BlockEntityTracingDistance #110

Closed
HowardZHY opened this issue May 18, 2023 · 7 comments

Comments

@HowardZHY
Copy link

No description provided.

@HowardZHY HowardZHY changed the title Split tracingDistance to EntityTracingDistance and BlockEntityTracingDistance [Enhancement] Split tracingDistance to EntityTracingDistance and BlockEntityTracingDistance May 18, 2023
@tr7zw
Copy link
Owner

tr7zw commented May 18, 2023

Why?

@tr7zw
Copy link
Owner

tr7zw commented May 18, 2023

Making the tracking distance larger is not really possible due to the cubic square law, so the only option would be to make it smaller. And why try to track one smaller than the other? What would be the benefit?

@HowardZHY
Copy link
Author

HowardZHY commented May 19, 2023

On some shooting server the view distance is 256 to sniper players & mobs but the map is complex.

@tr7zw
Copy link
Owner

tr7zw commented May 19, 2023

Again, setting it to 256 will just break it and not making it work at all.

@HowardZHY
Copy link
Author

the ideal settings might be
BlockEntityTracingDistance 32
EntityTracingDistance 240

@tr7zw
Copy link
Owner

tr7zw commented May 19, 2023

No. Anything above the default setting will cause issues. The issue is not for how many it's culling, but the maximum range.

@tr7zw
Copy link
Owner

tr7zw commented May 19, 2023

At 240 range, it will create a 3.4gb cache that needs to be reset 20 times per second in theory. Thats just a no go.

@tr7zw tr7zw closed this as not planned Won't fix, can't repro, duplicate, stale May 19, 2023
HowardZHY added a commit to HowardZHY/EntityCulling-1.8 that referenced this issue May 19, 2023
HowardZHY added a commit to HowardZHY/EntityCulling-1.8 that referenced this issue May 19, 2023
This reverts commit c53f494.
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

No branches or pull requests

2 participants