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

Менеджер зелий (Менеджер физики игрока?) #1412

Merged
merged 4 commits into from
Jul 29, 2024

Conversation

vladtcvs
Copy link
Contributor

@vladtcvs vladtcvs commented Jun 2, 2024

Описание PR:

Создает список зелий, действующих в данный момент на игрока, влияющих на физические параметры: speed, jump, gravity.

По каждому из параметров берем эффект последнего зелья, действующего на данный параметр

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

  1. Проверить что при приеме зелья, влияющего на физические параметры, появляется индикатор воздействия зелий
  2. проверить что при окончании действия всех зелий индикатор исчезает
  3. Проверить что при повторном приеме зелья эффект продлевается
  4. Проверить что при приеме более сильного или более слабого сила эффекта соответствует силе последнего
  5. Проверить выход и повторный заход игрока
  6. Проверить остановку сервера
  7. Проверить что при смерти эффект зелий исчезает

@vladtcvs
Copy link
Contributor Author

vladtcvs commented Jun 2, 2024

Замечние по коду: менеджер вынесен в отдельный мод lord_potion_effects потому что нам еще воздействие зелий на мобов прикручивать и там он тоже понадобится

@@ -0,0 +1,4 @@
lord_potion_effects = {}
Copy link
Member

Choose a reason for hiding this comment

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

  1. его бы привести к нашему общему формату модов.
    есть команда ./lord в корне проекта она создаёт болванку в этом формате.
    примеры реализаций в lord_rocks, lord_ground, lord_planks, ...

  2. давай сразу его назовём player_... (effects?, physics?, ...) и если он остаётся, как сейчас, вприципе независимым game-agnostic апи, то стоит его расположить в Core и без приставки lord_

  3. есть готовое гибкое решение https://github.com/minetest-mods/player_monoids . Посмотри на него плиз. Может стоит его заюзать ?

Copy link
Member

@alek13 alek13 left a comment

Choose a reason for hiding this comment

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

описал выше в комменте к моду.
требуется research https://github.com/minetest-mods/player_monoids
есть ещё надстройка поверх https://github.com/t-affeldt/minetest_playerphysics/blob/main/API.md

@alek13
Copy link
Member

alek13 commented Jun 5, 2024

@vladtcvs FYI: третий пункт можно вынести в отдельную задачи, а пока что поправить первые 2 и замерджить

@alek13 alek13 changed the title Менеджер зелий Менеджер зелий (Менеджер физики игрока?) Jun 5, 2024
@alek13 alek13 changed the base branch from dev to potions_manager July 29, 2024 00:11
@alek13 alek13 merged commit ee9a6d4 into lord-server:potions_manager Jul 29, 2024
alek13 added a commit that referenced this pull request Aug 24, 2024
alek13 added a commit that referenced this pull request Aug 24, 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.

2 participants