Remove World Borders, problem at positions 12500x+. Now it works#9
Open
KyuubiDDragon wants to merge 2 commits into
Open
Remove World Borders, problem at positions 12500x+. Now it works#9KyuubiDDragon wants to merge 2 commits into
KyuubiDDragon wants to merge 2 commits into
Conversation
The mod was compiled against the January HytaleServer.jar; the May 2026 build changed several APIs, which made the map all-black and stopped player markers. Recompiled against the new jar with these source fixes: API drift (shared root of "no players" + part of the black map): - com.hypixel.hytale.math.vector.Vector3d/Vector3f were removed. Transform now returns org.joml.Vector3d (position) and Rotation3f (rotation, yaw()). Swapped imports + types in PlayerTracker, PlayerHandler, EasyWebMapCommand, TileManager. Broadened PlayerTracker's scheduled-task catch to Throwable so a future Error can't silently kill the broadcast loop. - IChunkLoader.getIndexes() return type LongSet -> LongList; TileManager uses LongCollection now. Black map (primary cause): MapImage dropped its flat int[] data field for palette + bitsPerIndex + packedIndices. PngEncoder now unpacks that (toArgb) instead of reading the removed field. Rendering polish: - Don't cache empty/black tiles and auto-invalidate stale black placeholders on read (the byte-length empty check was useless: minimal compression makes even a solid-black tile ~197 KB). Map self-heals on view. - Switch the tile pipeline to ARGB: rendered terrain is opaque, unexplored / missing composite sub-chunks are transparent instead of opaque black, so zoomed-out composite gaps show the map background seamlessly. NOTE: requires lib/HytaleServer.jar updated to the 2026-05 build to compile (that 123 MB binary is intentionally kept out of this commit). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fix for #8