Skip to content

Реворк культа#6843

Merged
KIBORG04 merged 312 commits intoTauCetiStation:masterfrom
KIBORG04:cult_rework
Apr 10, 2021
Merged

Реворк культа#6843
KIBORG04 merged 312 commits intoTauCetiStation:masterfrom
KIBORG04:cult_rework

Conversation

@KIBORG04
Copy link
Member

@KIBORG04 KIBORG04 commented Feb 7, 2021

Для Работяг

Культ был переосмыслен в сторону меньшего хаоса и большей понятливости антивикичечников. В данным момент он может быть плохо забаланшен в цифрах, но это будет исправляться. Гайд только по ссылке ниже:
https://wiki.taucetistation.org/Участник:KIBORG/borer
А ещё вам я запрещаю выражать уникальное и очень важное ваше мнение под этим ПРом, нойте на форуме ссылаясь на этот ПР, пожалуйста. (контрибутеров это не касается)

Для комдеров

Впринципе, ПР готов к тестам, но надо ещё мне будет посмотреть чексы спройтов и карты.

Множество рефакторов(ритуалы, руны, ещё чё-то), новые фреймворки, один с тг на временные визуальные эффекты, а другое для культа.
Баги и рантаймы, скорее всего, ещё будут во время ОБТ, потому что всё найти во время ЗБТ почти невозможно, тратит много времени и не всегда возможно.

Так же, я понимаю что ревью в 202 файла на 8к добавлений и 4.5к удалений ОЧЕНЬ сложно, но я пытался некоторые вещи делать другими ПРами, которые изначально хотел сюда впихнуть, а некоторое всё равно пошло сюда, например фикс удаления худов у ливнувщих гостов, рефактор бимов с тг, иконки кишок для мап-едиторов, впринципе доработка альтернейт_апперансов(ещё одна) и многое другое.

Этот ПР скорее всего будет конфликтовать со всеми возможными ТМами, поэтому ТМить нужно будет другие ПРы аккуратно(ну ведь не мержить же ЭТО в первый день, пусть люди побегают, потестят, поноют, я пофикшу и только тогда уже мерж)

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

Морально старый и неудобный режим станет красивым и понятным
fix #6651
fix #6640
fix #6073
fix #5623
fix #3641

Поступающие ишуи во время тестов
fix #6941
fix #6943
fix #6953
fix #6955
fix #6983

Авторство

Код: Я, ТГ, специальная квалифицированная команда профессиональных тестеров с секретного дикорд сервера.
Спруйты и FX-художники: DedYarick, Udav, ТГ, парадизы.
Левел-дизайн: Cheburek.
Ведущий звукоинженер: BiQndy.

Чеинжлог

🆑 KIBORG, DedYarick, Udav, Cheburek, BiQndy, Тестеры

  • balance[link]: Реворк культа.
  • balance: Скелеты невоспреимчивы к химии и станам.
  • balance: Капеллан теперь в случаи неудачного деконверта очень сильно страдает, а сам деконверт не моментальный.
  • balance: Святая вода не деконвертит культистов, если те выбрали аспект Salutis.
  • balance: У капеллана в святой бутылке теперь обычная вода, вместо святой.
  • tweak: Очки капеллана и божок видят членов своей религии.
  • tweak: Админам добавлена кнопка в Fun, позволяющая говорить от имени бога членам религии.
  • ru: Множество переводов у всего, связанного с капелланом и культом.

Выпил талисманов
Новые обжекты
Например, уже закожена возможность запомнить одну руну и кастовать бесплатно
Зачаток механа захвата станции
Ритуалы с конкретным таргетом или с бакленым мобом не будут корректно работать
…уи, кишки+мясо+кровь+руки+ноги в хеавен, обновлен верб Write in blood, новые ритуалы
Copy link
Member

@volas volas left a comment

Choose a reason for hiding this comment

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

Выше Ниже пара незначительных комметриев, а так апрув. Как решить что готово - можешь мержить.

Comment on lines +134 to +135
sound = "bluspaceanom"
/datum/announcement/station/cult/capture_area/play(area/A)
Copy link
Member

Choose a reason for hiding this comment

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

ужс

Copy link
Member Author

Choose a reason for hiding this comment

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

Такое придумал перверт ниченезнаю

Comment on lines +8 to +9


Copy link
Member

Choose a reason for hiding this comment

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

ужс

Copy link
Member Author

Choose a reason for hiding this comment

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

Такое придумал людук ниченезнаю

Comment on lines +102 to +107
var/prev_key // There is a runtime when the index remains in list, but the type or other devilry disappears
for(var/K in alternate_appearances)
// ghost apperance qdeling in main apperance
if(K == "[prev_key]_observer")
continue
prev_key = K
Copy link
Member

Choose a reason for hiding this comment

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

Это нельзя хандлить в самом apperance? Или может оно там по таймеру должно удаляться.

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Я над этой фигней очень долго сидел. Вроде как, я даже на тг заходил смотреть, есть ли там этот рантайм(не помню результат).

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

  • Первая итерация
  • Удаляем апперанс стены
  • Удаляется гост-апперанс стены
  • Из листа с апперансами куклы убирается мейн апперанс без ссылок и нулл-штук
  • Гост-апперанс удаляется криво и остается в листе ВРОДЕ!!! имя апперанса = null
  • Вторая итерация по этому в прошлом гост-апперансу рантаймит
  • Других итераций нету

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Хз объяснит ли это хоть что-то, но вообщем да, тут все плохо костыль

Copy link
Member

Choose a reason for hiding this comment

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

Может, можно было бы удалять из списка в appearance/destroy, тут только вызывать qdel для них. Комментарий для затычки есть, кто нибудь потом проходя мимо может разобраться лучше.

@KIBORG04
Copy link
Member Author

Хочу замержить культ седня ближе к вечеру, поэтому у вас последний шанс что-то поменять

@KIBORG04
Copy link
Member Author

KIBORG04 commented Apr 10, 2021

Последний шанс на реформы, пока гитхаб чекает

@KIBORG04 KIBORG04 merged commit b69b08c into TauCetiStation:master Apr 10, 2021
@KIBORG04 KIBORG04 deleted the cult_rework branch April 10, 2021 17:37
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Apr 10, 2021
@TauKitty
Copy link
Contributor

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

https://forum.taucetistation.org/t/golosovanie-kak-vam-novyj-kult/35536/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment