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

Mountain spawner node #1329

Closed
wants to merge 2 commits into from
Closed

Conversation

vladtcvs
Copy link
Contributor

@vladtcvs vladtcvs commented May 14, 2024

Описание PR:

Нода, которая будет почанково создавать гору под собой, работая по ABM. Это даст отсутствие зависания сервера пока гора генерится

Рекомендации к тесту:

  1. Проверить что гора генерится полностью и в конце нода исчезает
  2. Проверить что мапген отрабатывает как надо для неинициализированных чанков
  3. Проверить создание биомов
  4. Проверить что не виснет сервер независимо от размера горы

Дополнительная информация:

Related: #1307

@vladtcvs vladtcvs marked this pull request as ready for review May 14, 2024 06:30
@vladtcvs vladtcvs requested review from alek13, Doloment and VanicGame May 14, 2024 06:30
@vladtcvs vladtcvs marked this pull request as draft May 14, 2024 06:41
@vladtcvs vladtcvs marked this pull request as ready for review May 14, 2024 06:46
@vladtcvs vladtcvs marked this pull request as draft May 14, 2024 07:10
@alek13
Copy link
Member

alek13 commented May 21, 2024

тоже черновик, закрываем пока что? или нужно на него глянуть ?

@vladtcvs
Copy link
Contributor Author

vladtcvs commented Jun 2, 2024

Нужно глянуть

@vladtcvs vladtcvs marked this pull request as ready for review June 2, 2024 12:32
Build mountain on ABM

Process mountan chunk by chunk
@vladtcvs
Copy link
Contributor Author

тоже черновик, закрываем пока что? или нужно на него глянуть ?

обновил

@alek13
Copy link
Member

alek13 commented Dec 23, 2024

@vladtcvs . Тут пропал ui.lua, и получается старый вариант использовать нельзя.
А он нам очень нужен. Поэтому то я и спрашивал это черновик или нет.
Сейчас это мерджить нельзя.
Я пока закрою PR. Его сильно дорабатывать нужно.

@alek13 alek13 closed this Dec 23, 2024
@alek13
Copy link
Member

alek13 commented Jan 22, 2025

@vladtcvs , решил продублировать тут, чтоб не потеряли:

С переработкой закончил.
Проект тут: https://github.com/orgs/lord-server/projects/36
Доп. хотелки тут: https://github.com/orgs/lord-server/projects/36/views/2 , туда нужно добавить задачу про ноду.
Формочку она (нода) будет открывать ту же, только обработчик свой подсовывать.
К Генератору (mountgen.Generator) наверняка понадобится свой Итератор (вместо mountgen.generator.ChunksIterator),
тогда к ним интерфейс понадобится, но пока я не продумывал это. А может и не Итератор нужен (итератор, наверное, то тот же, просто пошаговый с запоминанием на каком шаге остановился).
В этом моменте нужно подумать как лучше сделать.

нода, это теперь получается часть вот этой задачи https://github.com/orgs/lord-server/projects/36/views/2?pane=issue&itemId=94625516&issue=lord-server%7Clord%7C1937

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.

2 participants