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.

Sign up for GitHub

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

Jump to bottom

Новая раса для TG SS220 Вульпочки #267

Merged
merged 50 commits into from
Dec 25, 2024
Merged

Новая раса для TG SS220 Вульпочки #267

merged 50 commits into from
Dec 25, 2024

Conversation

Nlaysi
Copy link

@Nlaysi Nlaysi commented May 17, 2024 •
edited
Loading

About The Pull Request

ПР добавляют новую расу вульпканинов, доступную при создании персонажа

image

Игровые механики:

  • Возможность вынюхивать запахи людей, курение временно блокирует способность
  • Более высокий диапазон комфортных температур
  • Чувствительные ушки, получают больше урона но позволяют слышать шёпот
  • Белковая диета, может есть только мясо и некоторые молочные продукты
  • Быстрый метаболизм (модификатор голода х1.3)
  • Модификатор опьянения x2.5
  • Модификатор зарплаты 0.8

Спрайты взяты с https://github.com/Alecksohs/ParadiseSpriteEdits, c некоторыми доработками и исправлениями

Добавлен мутационный токсин, для превращения в вульпу: Делается в ксено, вколов в зелёный экстракт 5u Love

Why It's Good For The Game

На TG маленькое разнообразие рас, поэтому необходимо перенести расы с парадайза. Вульпкане одна из самых популярных и интересных рас с развитым лором и хорошим отыгрышем игроков. Добавление этой расы разнообразит игровой геймплей, создаст множество РП ситуаций и привлечет новых игроков.

image

Changelog

🆑
add: Новая раунд-стартовая раса "Вульпканин"
tweak: ДНК для вульп
add: Адаптивная одежда для расы "Вульпканин"
add: Мутационный токсин для превращения в вульпу
tweak: Печать хвоста и языка вульпы в limbgrower
sound: Некоторые звуки которые издают вульпы
/:cl:

@github-actions github-actions bot changed the title Новая раса для TG SS220 – Вульпочки 🦊 Новая раса для TG SS220 Вульпочки May 17, 2024
@AyIong
Copy link
Collaborator

AyIong commented May 17, 2024

Да заебали с этими вульпами...

@Nlaysi Nlaysi changed the title Новая раса для TG SS220 Вульпочки Новая раса для TG SS220 Вульпочки 🦊 May 17, 2024
@github-actions github-actions bot changed the title Новая раса для TG SS220 Вульпочки 🦊 Новая раса для TG SS220 Вульпочки May 17, 2024
PhantornRU
PhantornRU requested changes May 17, 2024
View reviewed changes
code/modules/mob/living/carbon/human/_species.dm Outdated Show resolved Hide resolved
code/modules/mob/living/carbon/human/_species.dm Outdated Show resolved Hide resolved
code/modules/mob/living/carbon/human/dummy.dm Outdated Show resolved Hide resolved
@PhantornRU
Copy link

Откуда портировал и почему не указал, если портировал?

Например сейчас я хочу заняться портированием с MonkeyStation. Поэтому по хорошему твой модуль нужно подготовить под будущий модуль, чтобы потом не пришлось всё переиначивать и переделывать. От того и вопрос, сам ли ты делал этот порт?
#223

В чате разработки в дискорде тебя нету, что плохо.

@Deultus

This comment has been minimized.

Sign in to view
@PhantornRU
Copy link

Вульпкане одна из самых "интересных" с "развитым лором и хорошим отыгрышем" "создаст множество РП ситуаций и привлечет новых игроков" Анегдот дня

а ты кто? Я не помню тебя в разработчиках. Гитхаб здесь не для кыков. Лучше бы помог перенести ништяки, коль умеешь им пользоваться.

@PhantornRU
Copy link

В общем. По ПРу.
Жду ответов по прошлым вопросам. Также сразу пометки:
Разбери всё по папкам, по возможности перенеси сразу ТАЯР, чтобы ты понял о чем речь и сразу сделал всё как полагается.
С модульностью поможем-объясним.

Обратись к Ащиа за плашкой и зайди к нам в кодерский канал, поможем разобраться что куда (да, мы заманиваем тебя в подвал)

Furrior
Furrior reviewed May 17, 2024
View reviewed changes
modular_bandastation/species/code/_defines.dm Outdated
#define DEFAULT_SPRITE_LIST "default_sprites"

#define HEAD_HAIR_VULPKANIN (1<<7)
#define HEAD_FACIAL_HAIR_VULPKANIN (1<<101)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Это переполненное число будет.

modular_bandastation/species/code/dna.dm Outdated
Comment on lines 16 to 18
for(var/blocknum in 1 to DNA_MODULAR_BLOCKS_COUNT)
. += L[blocknum] || random_string(GET_UI_BLOCK_LEN(blocknum), GLOB.hex_characters)
. = ..()
Copy link
Collaborator

Choose a reason for hiding this comment

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

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

modular_bandastation/species/code/dna.dm Outdated
Comment on lines 22 to 48
if(DNA_VULPKANIN_BODY_MARKINGS)
set_uni_feature_block(blocknumber, construct_block(GLOB.vulpkanin_body_markings_list.Find(features["vulpkanin_body_markings"]), GLOB.vulpkanin_body_markings_list.len))
if(DNA_VULPKANIN_HEAD_MARKINGS)
set_uni_feature_block(blocknumber, construct_block(GLOB.vulpkanin_head_markings_list.Find(features["vulpkanin_head_markings"]), GLOB.vulpkanin_head_markings_list.len))
if(DNA_VULPKANIN_HEAD_ACCESSORIES)
set_uni_feature_block(blocknumber, construct_block(GLOB.vulpkanin_head_accessories_list.Find(features["vulpkanin_head_accessories"]), GLOB.vulpkanin_head_accessories_list.len))
if(DNA_VULPKANIN_TAIL)
set_uni_feature_block(blocknumber, construct_block(GLOB.tails_list_vulpkanin.Find(features["tail_vulpkanin"]), GLOB.tails_list_vulpkanin.len))
if(DNA_VULPKANIN_TAIL_MARKINGS)
set_uni_feature_block(blocknumber, construct_block(GLOB.vulpkanin_tail_markings_list.Find(features["tail_markings"]), GLOB.vulpkanin_tail_markings_list.len))
if(DNA_VULPKANIN_FACIAL_HAIR)
set_uni_feature_block(blocknumber, construct_block(GLOB.vulpkanin_facial_hair_list.Find(features["vulpkanin_facial_hair"]), GLOB.vulpkanin_facial_hair_list.len))

/mob/living/carbon/human/updateappearance(icon_update = TRUE, mutcolor_update = FALSE, mutations_overlay_update = FALSE)
. = ..()
if(dna.features["vulpkanin_body_markings"])
dna.features["vulpkanin_body_markings"] = GLOB.vulpkanin_body_markings_list[deconstruct_block(get_uni_feature_block(dna.unique_features, DNA_VULPKANIN_BODY_MARKINGS), GLOB.vulpkanin_body_markings_list.len)]
if(dna.features["vulpkanin_head_markings"])
dna.features["vulpkanin_head_markings"] = GLOB.vulpkanin_head_markings_list[deconstruct_block(get_uni_feature_block(dna.unique_features, DNA_VULPKANIN_HEAD_MARKINGS), GLOB.vulpkanin_head_markings_list.len)]
if(dna.features["vulpkanin_head_accessories"])
dna.features["vulpkanin_head_accessories"] = GLOB.vulpkanin_head_accessories_list[deconstruct_block(get_uni_feature_block(dna.unique_features, DNA_VULPKANIN_HEAD_ACCESSORIES), GLOB.vulpkanin_head_accessories_list.len)]
if(dna.features["tail_vulpkanin"])
dna.features["tail_vulpkanin"] = GLOB.tails_list_vulpkanin[deconstruct_block(get_uni_feature_block(dna.unique_features, DNA_VULPKANIN_TAIL), GLOB.tails_list_vulpkanin.len)]
if(dna.features["tail_markings"])
dna.features["tail_markings"] = GLOB.vulpkanin_tail_markings_list[deconstruct_block(get_uni_feature_block(dna.unique_features, DNA_VULPKANIN_TAIL_MARKINGS), GLOB.vulpkanin_tail_markings_list.len)]
if(dna.features["vulpkanin_facial_hair"])
dna.features["vulpkanin_facial_hair"] = GLOB.vulpkanin_facial_hair_list[deconstruct_block(get_uni_feature_block(dna.unique_features, DNA_VULPKANIN_FACIAL_HAIR), GLOB.vulpkanin_facial_hair_list.len)]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Засунуть это в switch, чтоб не было повторных проверок

@Nlaysi
Copy link
Author

Nlaysi commented May 17, 2024

Откуда портировал и почему не указал, если портировал?

Например сейчас я хочу заняться портированием с MonkeyStation. Поэтому по хорошему твой модуль нужно подготовить под будущий модуль, чтобы потом не пришлось всё переиначивать и переделывать. От того и вопрос, сам ли ты делал этот порт? #223

В чате разработки в дискорде тебя нету, что плохо.

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

@Nlaysi
Copy link
Author

Nlaysi commented May 17, 2024

Вульпкане одна из самых "интересных" с "развитым лором и хорошим отыгрышем" "создаст множество РП ситуаций и привлечет новых игроков" Анегдот дня

а ты кто? Я не помню тебя в разработчиках. Гитхаб здесь не для кыков. Лучше бы помог перенести ништяки, коль умеешь им пользоваться.

а что бы вмержили, типа надо обязательно разрабом быть? не знал...

@larentoun
Copy link
Collaborator

Это реально??

Добавь в описание ПРа особенности расы. Если это просто рескин Хумана - аппрува не будет.

Добавь в описание ПРа то, откуда ты портировал или брал ассеты.

@VaskaPepelov
Copy link

Не надо вульп на тиге, по крайней мере без строгой привязки к их лору на уровне кода. Как выразился ХМЮ в дискорд канале, чтобы при выборе тех или иных кланов были уникальные причёски или цвета шкуры, и чтобы эти самые цвета были хоть сколько то натуральными.

"На TG маленькое разнообразие рас, поэтому необходимо перенести расы с парадайза. Вульпкане одна из самых популярных и интересных рас с развитым лором и хорошим отыгрышем игроков. Добавление этой расы разнообразит игровой геймплей, создаст множество РП ситуаций и привлечет новых игроков."

  1. На тг и вправду мало рас но в отличии от пары каждая крайне разнится от того же хумана значительно сильнее, этим тиге и вывозит.
    2)Да самая популярная, да её текущий лор действительно интересный и продуманный, но хороший отыгрышь? мы точно на одном сервере играем? чуть ли не две трети игроков на вульпах берут их потому что "Гыгыгыгы фури", при том даже не подозревают о каком то там лоре, и ещё обязательно выберут цвет меха как блевотно кислотно синий, или розовый, или зелёный( что кстати буквально показано на скрине в пре лол)
  2. Насчёт рп ситуаций, мрп - это инициатива от кого-то, следовательно хорошее рп идёт исключительно с подачи "хороших" и заряженных на рп игроков, как расписано в пункте два рассчитывать на это не стоит, банально потому что таких вытесняют рофлеры "гыгыгы фури, давай ты меня помоешь в душе а потом устроим жоскую лизню в дормах, я кстати капитан"
  3. новых игроков привлекает механ и стример а не раса с крайне сомнительной репутацией.

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

постскриптум

Вырежьте феленидов к едрене фене.

larentoun
larentoun requested changes May 22, 2024
View reviewed changes
Copy link
Collaborator

@larentoun larentoun left a comment

Choose a reason for hiding this comment

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

Думаю, что стоит убрать модификаторы урона, поднять порог холода и понизить порог жары.

Тебе ревью уже накидали, и вижу, что ты еще не закончил. Перевожу в драфт

@larentoun larentoun marked this pull request as draft May 22, 2024 10:48
@larentoun
Copy link
Collaborator

В орган легких дай повышенный резист к холоду и к жаре

В печени уменьши толерантность к алкоголю

@larentoun
Copy link
Collaborator

larentoun commented May 23, 2024 •
edited
Loading

Большие уши - большой урон по ушам

Ушам еще можно дать трейт TRAIT_GOOD_HEARING?

@larentoun
Copy link
Collaborator

Глаза чуть лучше видят в темноте (удаленный трейт) - tgstation#83356, значение LIGHTING_CUTOFF_REAL_LOW, но получают глаза, как у молей. Флэшбенг тайм.

Disclaimer: Не уверен, что стоит добавлять ночное зрение раундстартом

@larentoun
Copy link
Collaborator

Уменьши им зарплату до 0.8

@Nlaysi
Copy link
Author

Nlaysi commented May 23, 2024

Глаза чуть лучше видят в темноте (удаленный трейт) - tgstation#83356, значение LIGHTING_CUTOFF_REAL_LOW, но получают глаза, как у молей. Флэшбенг тайм.

Disclaimer: Не уверен, что стоит добавлять ночное зрение раундстартом

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

@Nlaysi Nlaysi force-pushed the vulpes branch 4 times, most recently from f6e137d to 6f8b6e1 Compare May 26, 2024 15:24
@Nlaysi
Merge remote-tracking branch 'upstream/master' into vulpes
3b66a88 
# Conflicts:
#	modular_bandastation/emote_panel/code/emotes.dm
@Nlaysi
Исправления и первый этап диги ножек
266f2d6
@Nlaysi
Merge remote-tracking branch 'upstream/master' into vulpes
3929ec6 
# Conflicts:
#	config/bandastation/bandastation_config.txt
#	modular_bandastation/modular_bandastation.dme
larentoun
larentoun previously requested changes Nov 27, 2024
View reviewed changes
Copy link
Collaborator

@larentoun larentoun left a comment

Choose a reason for hiding this comment

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

Ждём, пока и прошлые замечания по ревью завершишь

tgui/packages/tgui/interfaces/PreferencesMenu/preferences/features/species_features.tsx Outdated Show resolved Hide resolved
@dj-34
Copy link
Collaborator

dj-34 commented Dec 3, 2024

ну че там, долго еще?

Gaxeer
Gaxeer approved these changes Dec 25, 2024
View reviewed changes
Copy link
Collaborator

@Gaxeer Gaxeer left a comment

Choose a reason for hiding this comment

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

Лучше дочинить всем вместе, чем дрочить нляси вечно

@Gaxeer Gaxeer dismissed stale reviews from larentoun and PhantornRU December 25, 2024 13:19

Мержим похуй

@Gaxeer Gaxeer merged commit ce81e79 into ss220club:master Dec 25, 2024
20 of 21 checks passed
@Gaxeer Gaxeer mentioned this pull request Dec 25, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VaskaPepelov VaskaPepelov VaskaPepelov left review comments

@Deultus Deultus Deultus left review comments

@Furrior Furrior Furrior left review comments

@larentoun larentoun larentoun left review comments

@dj-34 dj-34 dj-34 left review comments

@Gaxeer Gaxeer Gaxeer approved these changes

@PhantornRU PhantornRU PhantornRU left review comments

Assignees

@larentoun larentoun

Labels
💾 Изменение конфига Ф-ф-фуриор...? 🔬 Testmerge Active Находится в тестмерже на сервере 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! RED LABEL ПР не может быть устаревшим. Но всем всё равно на него похуй. 🔉 Звук Нам скорее всего нравится как это звучит. 🛑 Do Not Merge Почему это висит лучше уточнить в дискорде. 220 проклятий тому кто зальёт это! TGUI Добавление или изменение существующего интерфейса на базе фреймворка TGUI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@Nlaysi @AyIong @PhantornRU @Deultus @larentoun @VaskaPepelov @dj-34 @Gaxeer @Furrior