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

Load heightmaps from anvil files #1917

Closed
wants to merge 2 commits into from

Conversation

connorrein
Copy link

Previously, heightmaps were hardcoded, which made it rain under surfaces:
2023-08-01_16 48 17

With this change, heightmaps are read from anvil files and it no longer rains under surfaces:
2023-08-01_16 46 08

In a future commit, the heightmaps will need to be updated on block placement.

mworzala added a commit to hollow-cube/minestom-ce that referenced this pull request Aug 5, 2023
Co-authored-by: tahmid-23 <60953955+tahmid-23@users.noreply.github.com>

See Minestom#1917
mworzala added a commit to hollow-cube/minestom-ce that referenced this pull request Sep 4, 2023
Co-authored-by: tahmid-23 <60953955+tahmid-23@users.noreply.github.com>

See Minestom#1917

(cherry picked from commit ef37e40)
mworzala added a commit that referenced this pull request Sep 8, 2023
Co-authored-by: tahmid-23 <60953955+tahmid-23@users.noreply.github.com>

See #1917

(cherry picked from commit ef37e40)
NxDs pushed a commit to NxDs/SakuraMinestom that referenced this pull request Oct 10, 2023
Co-authored-by: tahmid-23 <60953955+tahmid-23@users.noreply.github.com>

See Minestom#1917

(cherry picked from commit ef37e40)
mworzala added a commit that referenced this pull request Feb 9, 2024
Co-authored-by: tahmid-23 <60953955+tahmid-23@users.noreply.github.com>

See #1917

(cherry picked from commit ef37e40)
@reosfire
Copy link
Contributor

This logic also is impelemented here: #2044. If my pr will be merged this pr probably should be closed

@reosfire
Copy link
Contributor

@mworzala this pr can be closed

@connorrein connorrein closed this Apr 30, 2024
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.

None yet

2 participants