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

EnderIO Travel Anchor not properly culled when using Travel upgrade for EnderIO tools #18242

Open
3 tasks done
thundr opened this issue Dec 7, 2024 · 0 comments
Open
3 tasks done
Labels
Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed

Comments

@thundr
Copy link

thundr commented Dec 7, 2024

Your GTNH Discord Username

killurselfdiscor.d

Your Pack Version

2.6.1

Your Server

private

Java Version

Java 21

Type of Server

Vanilla Forge

Your Expectation

There is an issue present that was fixed in upstream here: SleepyTrousers/EnderIO-1.5-1.12#5125

See the related issue here: SleepyTrousers/EnderIO-1.5-1.12#4773

The Travel upgrade for EnderIO tools has an interaction with some graphics optimization mods that mess with frustum culling, causing the vector cast for finding a travel anchor to travel to to not check whether the anchor is along the vector in front or behind the player.

Ideally the player can only teleport to the travel anchor they are looking at. You can trivially reproduce this issue by placing a travel anchor behind the player and trying to shift right click. This will teleport the player backwards.

The Reality

With this bug, having a travel anchor behind the player counts as "selected" and will teleport you backwards.

Your Proposal

The implementation changes in the linked PR from upstream should be backported.

Final Checklist

  • I have searched this issue tracker and there is nothing similar already. Posting on a closed issue saying the bug still exists will prompt us to investigate and reopen it once we confirm your report.
  • I can reproduce this problem consistently by follow the exact steps I described above, or this does not need reproducing, e.g. recipe loophole.
  • I have asked other people and they confirm they also have this problem by follow the exact steps I described above, or this does not need reproducing, e.g. recipe loophole.
@thundr thundr added Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed labels Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Minor Status: Triage Issue awaiting triage. Remove once this issue is processed
Projects
None yet
Development

No branches or pull requests

1 participant