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

Удаление женского Displacement Map у ящеров #2814

Conversation

CaptainSqrBeard
Copy link
Contributor

@CaptainSqrBeard CaptainSqrBeard commented Nov 28, 2024

!!! Перед тем как мерджить !!!

Этот ПР на случай если женские дисплейсменты для ящеров не были сделаны по какой-либо причине
изображение

Описание PR

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

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

fixes #2808

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

нету

Медиа

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

Требования

Критические изменения

Список изменений

🆑 csqrb

  • fix: Унатхи женского пола теперь имеют старое отображение одежды на себе.

@lzk228
Copy link
Collaborator

lzk228 commented Nov 28, 2024

это не фикс
у женских ящеров другая форма тела и для них должны быть отдельные дисплейсменты

@CaptainSqrBeard
Copy link
Contributor Author

это не фикс
у женских ящеров другая форма тела и для них должны быть отдельные дисплейсменты

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

@lzk228
Copy link
Collaborator

lzk228 commented Nov 28, 2024

поясню тут
ДО #2447
у рептилий была только одна дисплейсмент карта, которая на женщинах убирала пару пикселей на костюмах в районе живота
ПОСЛЕ #2447
ко всем мужчинам применяется новая дисплейсмент карта которая учитывает ноги итд, к женщинам же всё ещё применяется старая карта

если говорить простыми словами, надо две этих карты соединить теперь

@lzk228 lzk228 closed this Nov 28, 2024
@CaptainSqrBeard
Copy link
Contributor Author

Разве этот баг не связан с добавлением в апстриме возможности использования отдельных ДМ для женщин?

@lzk228
Copy link
Collaborator

lzk228 commented Nov 28, 2024

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

@CaptainSqrBeard
Copy link
Contributor Author

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

а как оно раньше убирало пиксели для женщин, если той системы ещё не было?

@lzk228
Copy link
Collaborator

lzk228 commented Nov 28, 2024

маски какие-то накладывались, потом было переработано в space-wizards/space-station-14#30093
image

@lzk228 lzk228 added Changes: No C# Code has not been changed and removed No C# labels Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Code has not been changed size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

female digitigrade displacement map
2 participants