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

feat: VoidESP #5199

Open
wants to merge 5 commits into
base: nextgen
Choose a base branch
from
Open

Conversation

MukjepScarlet
Copy link
Contributor

VoidESP.mp4

Copy link
Contributor

@superblaubeere27 superblaubeere27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the use case?

Comment on lines +132 to +157
private val tickHandler = handler<PlayerTickEvent> {
val positions = search()

// Invalidate last tick positions
with(lastTickPositions.longIterator()) {
while (hasNext()) {
val longValue = nextLong()
if (longValue !in positions) {
renderer.removeBlock(mutable.set(longValue))
}
}
}

// Add this tick positions
with(positions.longIterator()) {
while (hasNext()) {
val longValue = nextLong()
renderer.addBlock(mutable.set(longValue))
}
}

lastTickPositions = positions

// Update cull data
renderer.updateAll()
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't do that. Look in BlockESP how it's done.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The BlockESP does not have animations for changes, so I asked @ccetl about how to use the PlacementRenderer

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't do that. Look in BlockESP how it's done.

BlockESP doesn't have animations nor culling which is especially important if the blocks disappear / appear in a short distance. But to some degree i agree, you don't have to introduce a separate list here as the renderer has in/current lists already. But IMO it's fine.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The BlockESP does not have animations for changes, so I asked @ccetl about how to use the PlacementRenderer

That is not the problem. You scan the region around the player every tick. This is inefficient.

Copy link
Contributor

@ccetl ccetl Jan 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah I didn't realize that you meant that, sorry.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The BlockESP does not have animations for changes, so I asked @ccetl about how to use the PlacementRenderer

That is not the problem. You scan the region around the player every tick. This is inefficient.

But in my profiler this doesn't spend long time to check. The HoleESP-like scanning might make the code more complicated

@MukjepScarlet
Copy link
Contributor Author

Like in bedwars the bridge was broken by someone and you are walking without notice of that, this can remind you of that

@MukjepScarlet
Copy link
Contributor Author

image
here

Copy link
Contributor

@sqlerrorthing sqlerrorthing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add more language translations

@ccetl
Copy link
Contributor

ccetl commented Jan 6, 2025

add more language translations

This should definitely be done by people who maintain the individual languages. If we just fill the translation files with gtranslate/deepl translations it will sound bad in many cases and then we could quit manually adding translations and just make a dynamic translation automation.

}

private val mutable = BlockPos.Mutable()
private val mutable1 = BlockPos.Mutable()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix naming please

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