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

Reflectors #76

Merged
merged 15 commits into from
Sep 22, 2024
Merged

Reflectors #76

merged 15 commits into from
Sep 22, 2024

Conversation

NikitosAseev
Copy link
Contributor

@NikitosAseev NikitosAseev commented Sep 1, 2024

О PR

Добавлены рефлекторы а именно "Коробка" , "Угловой" . "Двойной-Угловой"
Так же их изучение в РНД, крафт плат и самих рефлекторов

Почему / Баланс

Позволит придумывать различные схемы по поддержанию сингулярности.
РНД может использовать для безопасного дальнего взаемодействия с аномалией
Потенциально будет нужен для СверМатерии

Технические детали

Whitelist / Blacklist - указывать какие типы проджектайлов мы хотим отражать

BlockedDirections - при попадании в эту сторону отражение не будет происходить ( блокируется грубо говоря )

ReflectionDirection - куда будет отражен снаряд, по умолчанию он null так как в случае углового отражения - указывать сторону не нужно

ReflectorType - Simple и Angular ( два типа отражения, одиночный и угловой, для одиночного нужно еще будет указать сторону отражения ReflectionDirection )

Отражение происходит через смену локальной позиции и выстрел через GunComponent

Почему я использую систему выстрела оружий а не просто меняю поворот и координаты?
Ибо у снаряда есть линейная скорость помимо поворота и координат, с которой я не понимаю как нормально работать.

  • к тому же данная практика позволяет меньше щиткодить и использовать готовые методы
  • позволяет тонко настраивать отражение снаряда относительно возможностей Gun системы, если посмотреть то многие системы используют подобную практику, хоть и основаны они не на текущем снаряде, а на спавне его

Медиа

  • Я добавил к этому PR скриншоты/видео, демонстрирующие его изменения в игре, или этот PR не требует демонстрации в игре

Changelog

Copy link
Contributor

github-actions bot commented Sep 1, 2024

RSI Diff Bot; head commit fe9aa83 merging into fbbfb48
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Stories/Objects/Misc/module.rsi

State Old New Status
reflectorbox_circuitboard Added
reflectorcorner_circuitboard Added
reflectortsidecorner_circuitboard Added

Resources/Textures/Stories/Structures/Machines/reflectors.rsi

State Old New Status
reflector_circle Added
reflector_corner Added
reflector_corner_tside Added

Edit: diff updated after fe9aa83

@aristophanivan
Copy link
Contributor

Рефлекторы - круто. Освободившийся Дэн - еще лучше)

Copy link
Contributor

github-actions bot commented Sep 6, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

RobustToolbox Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Убери.

Copy link
Contributor

Choose a reason for hiding this comment

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

Убери.

runclient.sh Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Убери.

Copy link
Contributor

Choose a reason for hiding this comment

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

Убери.

runserver.sh Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Убери.

Comment on lines 104 to 106
- ReflectorBoxMachineCircuitboard
- ReflectorCornerMachineCircuitboard
- ReflectorCornerTsideMachineCircuitboard
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- ReflectorBoxMachineCircuitboard
- ReflectorCornerMachineCircuitboard
- ReflectorCornerTsideMachineCircuitboard
- ReflectorBoxMachineCircuitboard # Stories
- ReflectorCornerMachineCircuitboard # Stories
- ReflectorCornerTsideMachineCircuitboard # Stories

Comment on lines 599 to 613

- type: latheRecipe
parent: BaseCircuitboardRecipe
id: ReflectorBoxMachineCircuitboard
result: ReflectorBoxMachineCircuitboard

- type: latheRecipe
parent: BaseCircuitboardRecipe
id: ReflectorCornerMachineCircuitboard
result: ReflectorCornerMachineCircuitboard

- type: latheRecipe
parent: BaseCircuitboardRecipe
id: ReflectorCornerTsideMachineCircuitboard
result: ReflectorCornerTsideMachineCircuitboard
Copy link
Contributor

Choose a reason for hiding this comment

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

Перенеси в Stories папку.

Comment on lines 25 to 26
Steel: 50
Glass: 250
Copy link
Contributor

Choose a reason for hiding this comment

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

Не форматируй код оффов.

@NikitosAseev
Copy link
Contributor Author

@doublechest0 вроде все сделал

@doublechest0
Copy link
Contributor

Сделай новую технологию, которая будет в Stories папках для них.

У рефлекторов тир 2, стоимость изучения - 5000
AtmosphericTech -> ReflectionTech (Технологии отражения)

Исправил название коробочной продукции ( бадум тсс )
Comment on lines +521 to +523
- ReflectorBoxMachineCircuitboard # Stories
- ReflectorCornerMachineCircuitboard # Stories
- ReflectorCornerTsideMachineCircuitboard # Stories
Copy link
Contributor

Choose a reason for hiding this comment

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

Ты добавил новую технологию, но тут не убрал.

Copy link
Contributor Author

@NikitosAseev NikitosAseev Sep 21, 2024

Choose a reason for hiding this comment

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

это dynamicRecipes , иначе говоря при изучении технологии отражения у принтера схем открывается рецепт
без него при изучении технологии - крафта плат не будет

Copy link
Contributor

Choose a reason for hiding this comment

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

Понял.

@doublechest0 doublechest0 merged commit 26d5952 into Space-Stories:master Sep 22, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Выполнено
Development

Successfully merging this pull request may close these issues.

3 participants