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

Underused uplink stuff fix #1157

Merged
merged 6 commits into from
Aug 6, 2024

Conversation

TryHardo7
Copy link

@TryHardo7 TryHardo7 commented Jun 16, 2024

Данный фикс направлен на изменение тех позиций в аплинке которые практически никогда никто не покупает:
Раздел оружия:
Турель:
хп со 100(на 2 выстрела из дробовика) до 200
патроны - с 50 до 100
Коробка ножей - цена изменена с 6тк до 4тк, по 1 ножу за тк.

Раздел импланты:
Имплантер аплинк - цена изменена с 2тк до 1тк.
Имплантер побег - на момент его добавления, он телепортировал по всей станции, потом его нерфанули, теперь он
телепортирует на дистанцию от 0 до 7 тайлов. Формула расчёта берет данные из компонента грида и общей формулы
расчёта вообще всей математики в игре, туда я не лезу, но в сухом остатке мы получаем то, что 75%-80% всех
телепортов происходят в рамках от 0 до 5 тайлов, что часто даже из комнаты не выкидывает, потому что он проверяет
занят ли тайл и если да, кидает ближе к исходной точке. Увеличено кол-во телепортов с 2 до 12, так мы
получаем где-то 3-4 эффективных телепорта за 6тк. Его скорее всего и так покупать не будут, но для начала пойдет.
Имплантер ЕМП - изменено количество зарядов с 3 до 5.
Имплантер свобода - цена изменена с 5тк до 3тк.
Импланты после того как все заряды потрачены остаются в теле, таким образом чтобы купить новый и вставить, нужно раздобыть имплантер и вынуть старый потраченный, перед его заменой.

Раздел саботаж:
Кластерное мыло - цена изменена с 3тк до 2тк.

Раздел химия:
Сигареты - цена изменена с 2тк до 1 тк.

Раздел экипировка:
Рюкзак синдиката - размер внутреннего грида увеличен на 1 тайл(он был идентичен обычному рюкзаку).
Ева синдиката - цена изменена с 2тк до 1тк.
Магбутсы синдиката - убран штраф 10% к скорости бега, теперь их можно заправить(похоже, баг, что нельзя, ибо
интерфейс газтанка есть а функционала нет.

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 Lancevrot

  • tweak: Количество зарядов EMP-импланта повышено до 5.
  • tweak: Количество зарядов скрам-импланта повышено до 12.
  • tweak: Стоимость метательных ножей в аплинке изменена до 4 ТК.
  • tweak: Стоимость сигарет в аплинке уменьшена до 1 ТК.
  • tweak: Стоимость импланта освобождения в аплинке уменьшена до 3 ТК.
  • tweak: Стоимость EMP-импланта в аплинке уменьшена до 1 ТК.
  • tweak: Стоимость импланат-аплинка в аплинке уменьшена до 1 ТК.
  • tweak: Стоимость импланат-аплинка в аплинке уменьшена до 1 ТК.
  • tweak: Стоимость EVA в аплинке уменьшена до 1 ТК.
  • tweak: Объем баллона в магнитных ботинках синдиката увеличен до 5.
  • tweak: Урон турели синдиката увеличен до 200.
  • tweak: Боезапас турели синдиката увеличен до 100.

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Jun 16, 2024
# 2 minutes of thrust
volume: 0.75
# SS220 fix
volume: 5

Choose a reason for hiding this comment

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

Также вопрос в следующем - ты изменил объём, но не трогал кол-во вещества (те самые пресловутые моли), поэтому в магах изначально газа больше газа не станет. Это намеренно?

Copy link
Author

Choose a reason for hiding this comment

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

Да, я подумал что слишком жирно будет их сразу полностью заправленными делать, придётся найти канистру и заправить. Это всё ещё делает их полезнее, просто от игрока ещё действия потребуются чтобы их потенциал раскрыть

@gogenych
Copy link

@Ady4ik на одобрение

@Ady4ik
Copy link

Ady4ik commented Jun 19, 2024

Я думаю, надо на тесты будет закинуть

@TryHardo7
Copy link
Author

Я думаю, надо на тесты будет закинуть

Что значит на тесты закинуть, что для этого надо сделать?

Copy link

github-actions bot commented Jul 2, 2024

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

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Jul 2, 2024
@UrPrice
Copy link
Collaborator

UrPrice commented Jul 2, 2024

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

А можно чуть подробнее? Появился какой-то лимит имплантов? Теперь нельзя иметь какие-то импланты одновременно? Или это касается только однотипных имплантов? В реалиях последовательного передоза имплантами, насколько это вообще удобно, когда в тебе итак 4 импланта условно?

@TryHardo7
Copy link
Author

А можно чуть подробнее?

2 одинаковых импланта не ставятся одновременно. => пустой фридом нужно вынуть а потом кольнуть полный.

@gogenych
Copy link

gogenych commented Jul 2, 2024

@OwlMe1steR need test

@OwlMe1steR
Copy link

Итоги теста

Раздел оружия

Всё работает, как и написано в фиксе.

Раздел импланты

  1. Имплантер аплинк по-прежнему стоит 2 ТК.
  2. Имплантер побега один раз телепортировал в стену, в остальном работает стабильно, как и написано в фиксе.

Раздел саботаж

Всё работает, как и написано в фиксе.

Раздел химия

Всё работает, как и написано в фиксе.

Раздел экипировка

Всё работает, как и написано в фиксе, за исключением магбутсов синдиката.
Магбутсы синдиката не показывают бафы и дебафы (отсутствует иконка). Попытался посмотреть по компонентам и сравнить с нынешней релизной версией на сервере, однако все компоненты и их значения одинаковые.

Исправление стоимости аплинк имплентера.
@TryHardo7
Copy link
Author

  1. Имплантер аплинк по-прежнему стоит 2 ТК.
    Всё работает, как и написано в фиксе, за исключением магбутсов синдиката. Магбутсы синдиката не показывают бафы и дебафы (отсутствует иконка). Попытался посмотреть по компонентам и сравнить с нынешней релизной версией на сервере, однако все компоненты и их значения одинаковые.

Исправил имлантер аплинк, теперь стоимость 1ТК.
А касательно магбутсов я не понял о каких иконках речь. Нет иконки которая за включение эффекта ботинок отвечает? Или та, что справа на экране возле здоровья? Там должно отображаться только статус включения магнитной функции, не полёта.

@OwlMe1steR
Copy link

Исправил имлантер аплинк, теперь стоимость 1ТК. А касательно магбутсов я не понял о каких иконках речь. Нет иконки которая за включение эффекта ботинок отвечает? Или та, что справа на экране возле здоровья? Там должно отображаться только статус включения магнитной функции, не полёта.

Я про эту иконку "Изучить показатели скорости одежды". Она не отображалась, когда я запускал фикс на локалке, но отображалась, когда я попросил протестировать для меня на сервере. Поэтому я и не смог сравнить, убралось значение скорости бега или нет.

image

@TryHardo7
Copy link
Author

Я про эту иконку "Изучить показатели скорости одежды". Она не отображалась, когда я запускал фикс на локалке, но отображалась, когда я попросил протестировать для меня на сервере. Поэтому я и не смог сравнить, убралось значение скорости бега или нет.

Так она и не должна отображаться, модификаторов нет.

@OwlMe1steR
Copy link

Так она и не должна отображаться, модификаторов нет.

Тогда это значит, что и штраф 5% при ходьбе тоже убрали. Если так, то всё работает как и должно быть.

@gogenych
Copy link

gogenych commented Aug 6, 2024

@DexlerXD Тут застряло на месяц. Уже всё одобрено и потещенно

@gogenych
Copy link

gogenych commented Aug 6, 2024

@DexlerXD Тут застряло на месяц. Уже всё одобрено и потещенно

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

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Aug 6, 2024
@DexlerXD DexlerXD merged commit 1db696e into SerbiaStrong-220:master Aug 6, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants