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

[WiP] Респрайт всех рас #13686

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Slavik2001
Copy link
Contributor

@Slavik2001 Slavik2001 commented Dec 23, 2024

Описание изменений

Респрайт всех рас
Добавлена окраска хвоста для ящеров/унатхов

До:
image
image
image

После:
![image](image

Покраска:
image
image
image

image

Почему и что этот ПР улучшит

Красиво

Авторство

Я - код, @WatlerJe - спрайты

Чеинжлог

🆑 Slavk2001, WatlerJe

  • image: Перерисованы все расы.
  • image: Унатхи/ящеры теперь могут раскрасить хвост.

@TauKitty
Copy link
Contributor

Changelog status: 🔘 No changelog

@TauKitty TauKitty requested a review from a team December 23, 2024 05:17
@Slavik2001
Copy link
Contributor Author

Единственное но у покраски хвоста, из-за смены роста куклы ящера хвост остается неизменным и лишь опускается/поднимается на 1 пиксель из-за чего может быть оутлайн у низкой куклы.

У меня пока не получилось допереть что с этим сделать, а так ПР готов к мержу

@Chip11-n
Copy link
Contributor

Chip11-n commented Dec 23, 2024

Плувийцы, воксы и СПУ явно стали лучше, а вот остальные - спорно, на мой вкус. В особенности скреллы. Люди выглядят прикольно, но больно уж непривычно. Ну, а ящерицы - это ящерицы.

@Slavik2001
Copy link
Contributor Author

Ты еще не видел их всех в разных вариациях роста куклы. От там начинается веселуха XD

@TauKitty
Copy link
Contributor

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/resprajt-vseh-igrovyh-ras/44224/1

@ZER0-In
Copy link

ZER0-In commented Dec 23, 2024

Помню оставлял один реквест Вальтеру по поводу кастомизируемых конечностей для СПУ. Все уперлось в то, что нужен кодер.

Хотел бы узнать, ждать ли это в будущем/сейчас?

Спрайты мне нравятся. Будет прикольно поиграть, даже хотя бы в ТМе с ними.

Но хотел бы подметить, что на скринах хуман в ботинках. Помню там была маленькая проблема со спрайтами в процессе работы с ними по поводу ног (ступней), лишний пиксель затемнения. Интересно узнать, изменилось ли что-то.

@ZER0-In
Copy link

ZER0-In commented Dec 23, 2024

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

@DarkFurrSss
Copy link
Contributor

Челюсть унатхов охренеть стремно вышла

@maleyvich
Copy link
Contributor

17350172961405175090240383664503

буквально новые унатхи (все равно пиздато плюс хи-хи-хи)

@DarthSidiousPalpatine
Copy link
Contributor

У меня пока не получилось допереть что с этим сделать, а так ПР готов к мержу

Просто вставляй хвост после того как прорисуется голова...
Или вставляй хвост сразу со смещением по росту.

Крч надо немного обновить код рисования хумана.

@DarthSidiousPalpatine
Copy link
Contributor

DarthSidiousPalpatine commented Dec 24, 2024

Ещё проблема в том что гномирование опустит ещё ниже хвост и если его поднять выше - то будет странно смотреться со спины, допустим. Хвост будет расти из центра спины :/

В идеале бы для гномированных сделать отдельный спрайт маленького хвостика...

Или отрывать хвост целиком при гномировании. Дескать, отпал. После чего выдавать его итемом на землю под гномом.

@WatlerJe
Copy link
Contributor

WatlerJe commented Dec 24, 2024

Ещё проблема в том что гномирование опустит ещё ниже хвост и если его поднять выше - то будет странно смотреться со спины, допустим. Хвост будет расти из центра спины :/

В идеале бы для гномированных сделать отдельный спрайт маленького хвостика...

Или отрывать хвост целиком при гномировании. Дескать, отпал. После чего выдавать его итемом на землю под гномом.

Можно просто resize = 0.5 и дебафы накидывать всякие. Потому что текущие "Гномы" это обычные карланы.

@volas volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Dec 25, 2024
Comment on lines +169 to +174
if(owner.s_tone >= -75 && owner.s_tone <= 34)
black = null
else if(owner.s_tone >= -185 && owner.s_tone <= -74)
black = "_b"
else
black = null
Copy link
Contributor

Choose a reason for hiding this comment

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

это очень странная логика, которая:
а) не учитывает наличие флага HAS_SKIN_TONE
б) зачем нам хранить скин тон в цифрах, если это все равно в выбор между двумя цветами превращается

Copy link
Contributor Author

@Slavik2001 Slavik2001 Dec 25, 2024

Choose a reason for hiding this comment

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

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

Формат цыфр сохрянил чтобы не переделывать абсолютно всю систему и на всякий случай если Вальтер сделает еще несколько видов цвета кожи, но пока только 2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Или лучше черный цвет кожи кинуть в отдельный файл и если с_тон выполняется, то менять путь к файлу?

@DarkFurrSss
Copy link
Contributor

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

@simb11
Copy link
Contributor

simb11 commented Dec 26, 2024

человечки курят сигареты шеей
image

@WatlerJe
Copy link
Contributor

человечки курят сигареты шеей

Курят ртом, точка это тень под носом.

@simb11
Copy link
Contributor

simb11 commented Dec 26, 2024

морда унатхов сквозь большинство масок торчит

@WatlerJe
Copy link
Contributor

морда унатхов сквозь большинство масок торчит

Например каких?

@simb11
Copy link
Contributor

simb11 commented Dec 27, 2024

челюсть унатха с видом сбоку торчит сквозь все маски (имеется ввиду всё что суётся в слот маски), кроме гражданского и охранного противогаза

@WatlerJe
Copy link
Contributor

челюсть унатха с видом сбоку торчит сквозь все маски (имеется ввиду всё что суётся в слот маски), кроме гражданского и охранного противогаза

Скажу по секрету она и раньше торчала, так как все маски под людин нарисованы.

@volas volas removed the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Dec 27, 2024
@Tap0r
Copy link
Contributor

Tap0r commented Dec 27, 2024

ля а сделайте унатхам выбор челюги ака ящер/крокодил

@DarkFurrSss
Copy link
Contributor

Где респрайт дион? Раз всех рас

@WatlerJe
Copy link
Contributor

Где респрайт дион? Раз всех рас

Там же где и игроки на дионах.

@DarkFurrSss
Copy link
Contributor

Где респрайт дион? Раз всех рас

Там же где и игроки на дионах.

странно, в таком случае не вижу спрайтов на сервере

@hwat1204
Copy link

hwat1204 commented Jan 7, 2025

у меня картинка "после" не грузится, чё там?

@Slavik2001 Slavik2001 changed the title Респрайт всех рас [WiP] Респрайт всех рас Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.