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.

Sign up for GitHub

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

Jump to bottom

Горный посох может вызвать OOM #425

Closed
vladtcvs opened this issue Sep 5, 2021 · 0 comments · Fixed by #537
Closed

Горный посох может вызвать OOM #425

vladtcvs opened this issue Sep 5, 2021 · 0 comments · Fixed by #537
Assignees
@vladtcvs
Labels
admin side В большей степени имеет влияние на команду разработчиков и билдеров, чем на игроков bug Для задач: ошибка, которую необходимо исправить
Milestone
2022.02.p3

Comments

@vladtcvs
Copy link
Contributor

vladtcvs commented Sep 5, 2021

Сейчас горный посох загружает целиком блок карты, куда помещает гору, и при больших горах это может вызвать OOM и падение minetest. Надо править карту чанками (или бОльшими блоками)
@vladtcvs vladtcvs added the bug Для задач: ошибка, которую необходимо исправить label Sep 5, 2021
@Doloment Doloment added the admin side В большей степени имеет влияние на команду разработчиков и билдеров, чем на игроков label Jan 7, 2022
@vladtcvs vladtcvs mentioned this issue Feb 11, 2022
Merged
alek13 pushed a commit that referenced this issue May 26, 2022
@vladtcvs
Fixes mountain tool (#537, Closes #425)
a0d4162 
* Build explicit height map
* Write data with voxelmanip, but use blocks 64x64x64
* Check for y >= 1 when build mountain
* Set 0 at perimeter
@alek13 alek13 added this to the 2022.02.p3 milestone May 26, 2022
@alek13 alek13 closed this as completed May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vladtcvs vladtcvs

Labels
admin side В большей степени имеет влияние на команду разработчиков и билдеров, чем на игроков bug Для задач: ошибка, которую необходимо исправить
Projects
None yet
Milestone
2022.02.p3
Development

Successfully merging a pull request may close this issue.

Mountain vladtcvs/lord
3 participants
@alek13 @vladtcvs @Doloment