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

Cigarettes rework #1919

Merged
merged 32 commits into from
Oct 2, 2024
Merged

Cigarettes rework #1919

merged 32 commits into from
Oct 2, 2024

Conversation

TryHardo7
Copy link

@TryHardo7 TryHardo7 commented Sep 23, 2024

Создал несколько новых уникальных для некоторых отделов и не только сигарет со стилизованными пачками.
Создал новую базу для визуалайзера сигарет внутри пачки для новых пачек в перспективе 3х4 а не кабинета.
Старые остались как были, мне лень их перерисовывать и вообще я не спрайтер.

image

Был добавлен новый химикат - Неоникатинойд, менее вредный для гуманоидов при курении.
Этот химикат лежит в основе не-лечащих сигарет, нужен для баланса генетического урона, потому что в лечащих
сигаретах меньше вещества, а в оказывающих эффект больше.

Разделил все сигареты в игре на 2 фундаментальные базы - лечащие и оказывающие эффект, в лечащих сигаретах никотин, в оказывающих эффект - неоникатиноид.
Курение переделано так, что теперь оно заносит в организм вещества из сигарет со скоростью более близкой к скорости метаболизма, так реагенты не успевают копиться внутри куклы, но успевают усвоиться и оказать эффект.

Теперь курение вызывает рак, приблизительно 1 сигарета наносит 1-1.3 клеточного урона

Время курения сигарет до реворка и после примерно сровнял, для игрока разницы не заметно.

Добавил локализацию для всех сигарет.
Добавил новые сигареты, посвященные отделам в вендоры соответствующих отделов.

Никотин в шлёпских сигаретах заменён на неоникатинойд, чтобы шлёпстеры поменьше травились.

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

🆑 Lancevrot

  • add: Добавлены новые виды сигарет!
  • add: Курение вызывает рак! Ура!
  • tweak: Старым сигаретам добавлены новые эффекты!
  • fix: Исправлено - эффекты химии в сигаретах не срабатывали при курении!

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Sep 23, 2024
Copy link

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

@github-actions github-actions bot added Changes: Sprites Изменение затронуло файлы ".rsu / .png" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" and removed Merge Conflict Some content has conflicts. labels Sep 23, 2024
Copy link

github-actions bot commented Sep 23, 2024

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

Resources/Textures/SS220/Objects/Consumable/Smokeables/Cigaretts/Packs/AVD.rsi

State Old New Status
closed Added
inhand-left Added
inhand-right Added
open Added

Resources/Textures/SS220/Objects/Consumable/Smokeables/Cigaretts/Packs/ENG.rsi

State Old New Status
closed Added
inhand-left Added
inhand-right Added
open Added

Resources/Textures/SS220/Objects/Consumable/Smokeables/Cigaretts/Packs/MED.rsi

State Old New Status
closed Added
inhand-left Added
inhand-right Added
open Added

Resources/Textures/SS220/Objects/Consumable/Smokeables/Cigaretts/Packs/RnD.rsi

State Old New Status
closed Added
inhand-left Added
inhand-right Added
open Added

Resources/Textures/SS220/Objects/Consumable/Smokeables/Cigaretts/Packs/SEC.rsi

State Old New Status
closed Added
inhand-left Added
inhand-right Added
open Added

Resources/Textures/SS220/Objects/Consumable/Smokeables/Cigaretts/Packs/WEH.rsi

State Old New Status
closed Added
inhand-left Added
inhand-right Added
open Added

Resources/Textures/SS220/Objects/Consumable/Smokeables/Cigaretts/Packs/visualizer.rsi

State Old New Status
1 Added
10 Added
2 Added
3 Added
4 Added
5 Added
6 Added
7 Added
8 Added
9 Added

Edit: diff updated after 09f7e91

Copy link

@gogenych gogenych left a comment

Choose a reason for hiding this comment

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

Всё гуд. Спрайты бы Кемрану показать

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Sep 23, 2024
Copy link

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

Copy link

@Ady4ik Ady4ik left a comment

Choose a reason for hiding this comment

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

Посмотрел только поверхностно, увынск

TryHardo7 and others added 12 commits September 23, 2024 22:59
added comments
@TryHardo7
Copy link
Author

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

@Ady4ik Ady4ik changed the title Cigrettes rework [DNM] Cigrettes rework Sep 24, 2024
@TryHardo7 TryHardo7 changed the title [DNM] Cigrettes rework Cigarettes rework Sep 28, 2024
@EstKemran
Copy link

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

Добавь эту картинку в описание ПРа

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Sep 29, 2024
Copy link

@Ady4ik Ady4ik left a comment

Choose a reason for hiding this comment

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

after requested changes rfm

@Ady4ik Ady4ik self-requested a review October 1, 2024 14:16
@DexlerXD
Copy link

DexlerXD commented Oct 1, 2024

чекам всё ещё плохо...

@Ady4ik Ady4ik merged commit e9c55d1 into SerbiaStrong-220:master Oct 2, 2024
15 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" Changes: Sprites Изменение затронуло файлы ".rsu / .png" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants