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

Новый декор. #12655

Conversation

DarthSidiousPalpatine
Copy link
Contributor

@DarthSidiousPalpatine DarthSidiousPalpatine commented Dec 30, 2023

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

Добавил:
Глобус Венеры, Земли, Яргона-4:
image

Деревянные настольные часы:
image

Абстрактные статуэтки:
image

Полки:
image
image
image
image
image

3 бюста должностных лиц НТ с подписями (спасибо WhalePower):
image
image
image
image

15 ваз для цветов:
image
image
image
image
image
image
image
image

4 вида стикеров:
image

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

Как выглядят глобусы, статуэтки, настольные часы и бюсты в игре (Я их расставил в разных местах крч, где подумал что будет норм там и расставил):
image
image
image
image
image

Как выглядят вазы в игре (тоже расставил где подумал прикольно):
image

Как выглядят стикеры в игре:
image
image

Авторство

AndreyGysev и все все все, кто помогал в дискорде.
WhalePower написал имена и описания для бюстов.

Чеинжлог

🆑 AndreyGysev, WhalePower

  • rscadd: Множество декоративных вещей
  • rscadd: Стикеры

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Map Edit Sprites labels Dec 30, 2023
@TauKitty TauKitty requested review from a team December 30, 2023 09:00
@TauKitty
Copy link
Contributor

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

https://forum.taucetistation.org/t/fludilka-chast-2/28659/30043

Copy link
Contributor

@WatlerJe WatlerJe left a comment

Choose a reason for hiding this comment

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

Основная претензия на которой я остановлюсь это - Все в одном ПРе.
Будет лучше разнести фичи по разным ПРам и уже там оценивать пригодность по спрайтерскому признаку.

@DarthSidiousPalpatine
Copy link
Contributor Author

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

Оно всё имеет минимальный механ и чисто декоративное.

@WatlerJe
Copy link
Contributor

WatlerJe commented Dec 30, 2023

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

@DarthSidiousPalpatine
Copy link
Contributor Author

Ну опиши недостатки по спрайтам и я их заменю, хз.

@WatlerJe
Copy link
Contributor

Я уже написал, что первый и главный это то что все в одном ПРе.

@DarthSidiousPalpatine
Copy link
Contributor Author

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

@DarthSidiousPalpatine
Copy link
Contributor Author

То что спрайты в одном ПРе это не недостаток спрайтов.

@arygal
Copy link
Contributor

arygal commented Dec 30, 2023

Холодосы в отделах как то вырвиглазно смотрятся. Может стоит покрасить его в нейтральный\цвет отдела.

@DarthSidiousPalpatine
Copy link
Contributor Author

DarthSidiousPalpatine commented Dec 30, 2023

Холодосы в отделах как то вырвиглазно смотрятся. Может стоит покрасить его в нейтральный\цвет отдела.

Это же фирменные холодосы Einstein Electronics, они должны быть красными :P
image

@DarthSidiousPalpatine
Copy link
Contributor Author

Холодосы в отделах как то вырвиглазно смотрятся. Может стоит покрасить его в нейтральный\цвет отдела.

Я ещё потом когда-нибудь микроволновки перерисую :dovolen:

@TauKitty TauKitty requested a review from a team December 31, 2023 02:48
@DarthSidiousPalpatine
Copy link
Contributor Author

Всё нормально, добавляю цветные микроволновки, кулеры и тостеры

@DarthSidiousPalpatine
Copy link
Contributor Author

Чтобы холодильники цветные вписались

@DarthSidiousPalpatine
Copy link
Contributor Author

Обновил ПР, обновил описание, картинок добавил ещё больше.

@DarthSidiousPalpatine
Copy link
Contributor Author

Важный несёт бутылку
image

@DarthSidiousPalpatine
Copy link
Contributor Author

Обновил статуэтки, сделал полки в цвете столов, обновил превью ПРа.

@DarthSidiousPalpatine
Copy link
Contributor Author

Убрал аутлайн у полки.

Добавил хуешифтинга мб. Мб лучше стало, мб нет. Обновил превью ПРа.

Copy link
Contributor

@Deahaka Deahaka left a comment

Choose a reason for hiding this comment

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

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

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

Copy link

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@volas
Copy link
Member

volas commented Feb 17, 2024

Стикеры, полки, настольные глобусы, бюсты и 4 фигурки можно добавлять

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

@DarthSidiousPalpatine
Copy link
Contributor Author

Стикеры, полки, настольные глобусы, бюсты и 4 фигурки можно добавлять

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

Рисовал я, а зальют другие. Ну-ну. Нахуй написал в Stalled PR-е? Теперь ещё 4 недели ждать закрытия.

@DarthSidiousPalpatine
Copy link
Contributor Author

DarthSidiousPalpatine commented Feb 17, 2024

Что тут кроме "стикеров, полок, глобусов, бюстов и 4 фигурок" интересн?

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

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

@volas
Copy link
Member

volas commented Feb 18, 2024

тебя изначально просили разбить на отдельные пр-ы, твои возмущения не понятны.

@DarthSidiousPalpatine
Copy link
Contributor Author

DarthSidiousPalpatine commented Feb 18, 2024

тебя изначально просили разбить на отдельные пр-ы, твои возмущения не понятны.

Вальтер такой: Пихает в один ПР респрайт ящика для руды, пулевых отверстий стен и теней для столов
Волас: "Всё в одной тематике ПРа, всё норм, мерж"

Гусев такой: Пихает в один ПР 5 декор-вещей настольных
Волас: "Гусев ну ты дебил всё разное разбей ПР, кто-нибудь сделайте такой же ПР но без напольных глобусов и мерж"

@DarthSidiousPalpatine
Copy link
Contributor Author

Особенно топ, что респрайт столов с тенями под ними не позволял ударить челика под столом :dovolen:

@volas
Copy link
Member

volas commented Feb 18, 2024

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

@DarthSidiousPalpatine
Copy link
Contributor Author

сразу спрайтеры забраковали

Да я понял уже что удава ты и вальтер за спрайтера не считаете...

@DarthSidiousPalpatine
Copy link
Contributor Author

Напомню, в отличие от вальтера, удав не побрезговал сделать нормальный ревью почти сразу :dovolen:

@DarthSidiousPalpatine
Copy link
Contributor Author

ПР на 14 ликасов, супер-топ, все мапперы ждут декор, но Вальтеру же не нравится идея иметь настольные часы, уууу

@Tap0r
Copy link
Contributor

Tap0r commented Feb 19, 2024

ПР на 14 ликасов, супер-топ, все мапперы ждут декор, но Вальтеру же не нравится идея иметь настольные часы, уууу

Маперы мертвы
ssIVJ1k_xlc.jpg

@DarthSidiousPalpatine
Copy link
Contributor Author

Маперы мертвы

Даже мапперы на тау умерли теперь... Жесть. А Револь не вернулся гамму апдейтить?

@Tap0r
Copy link
Contributor

Tap0r commented Feb 20, 2024

Револь ушел за сигаретами и не вернулся

@DarthSidiousPalpatine
Copy link
Contributor Author

За год единственное что добавилось на Тау Кита нового - репликаторы от Людука. Больше никаких новых режимов, новых механик, ничего. Респрайты от Вальтера - полно. Фиксы багов - полно. ПРы а-ля "передвинул лампочку в бриге" - ещё больше. Даже балансоправок ебейшее количество. Но ничего нового не добавляется :dovolen:

Ясен пень, что нового добавить, когда даже 5 декоративных настольных предметов добавить 2 полных месяца не могут.
А с Погребком, новой механикой, что своей подсистемой открыла бы дорогу паре-тройке других моих ПРов - всё ахуеть как сложно, ведь надо сначала мозги выебать "Гисев сделай вот так с датумами", потом мозги выебать "ПР аппрувнут, тебе осталось только рецепты вернуть", а когда вернул - снова мозги начать ебать "нет гисев ты сделал всё не так, вообще ничего не делай я сам через 3 года по решению своей левой пятки, возможно, сделаю похожий механ и тогдаааааааа".

Надо только подождать, там всё будет бесплатно, там всё будет в кайф. Там, наверное, вообще будет онлайн под 60!

@volas
Copy link
Member

volas commented Feb 22, 2024

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

@DarthSidiousPalpatine
Copy link
Contributor Author

DarthSidiousPalpatine commented Feb 22, 2024

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

Та лол, я сразу понял что ты просто напросто никогда не сделаешь континуити. Мне как-то пофиг)

Copy link

github-actions bot commented Mar 7, 2024

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@AzzyDreemurr13 AzzyDreemurr13 added Merge Conflict Awaiting Author Требуется присутствие автора labels Mar 12, 2024
Copy link

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

@github-actions github-actions bot closed this Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Author Требуется присутствие автора Awaiting Review [Sprite Dep] Ожидается ревью от спрайт департамента Feature Новая фича Map Edit Merge Conflict Sprites Stalled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.