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

xenoarchaeology part code readability, some artifact changes #7933

Merged
merged 47 commits into from Sep 14, 2021
Merged

xenoarchaeology part code readability, some artifact changes #7933

merged 47 commits into from Sep 14, 2021

Conversation

Morar1
Copy link
Contributor

@Morar1 Morar1 commented Sep 8, 2021

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

Убрано
Большинство рандомных проценты в активации эффектов артефактов.
Рандомные проценты для активации артефактов.
Триггер артефакта от спрайта.
Добавлено
Очки за отчеты артефактов: 10к поинов за мейн эффект, 5к поинтов за вторичный эффект.
Мудлеты для артефактов воздействующих посылающих плохие/хорошие сообщения
Бутылка воды со случайными реагентами в находки.
Уничтоженных мехи в находки.
Стоимость на активацию эффекта артефакта, аура тратит весь заряд артефакта, тронуть тратит 3, аура ничего не стоит.
Сила некоторых эффектов увеличивается/уменьшается от потраченного заряда.
Некоторые эффекты артефактов уменьшаются с увеличением защиты от артефактов.
Эффект артефакта: гравитация. Притягивает или отдаляет моба.
Эффект артефакта: свет. Дает свет, либо удаляет его рядом.
Эффект артефакта: звук. Воспроизводит звуки.
Артефакты можно уничтожать.
Воздействие при уничтожении артефакта для некоторых типов эффектов.

Исправлено
Отчеты артефактов дают очки рнд сканируйте отчет science tool-ом, и загрузите в консоль изучений
Говорящая маска не заставляет говорить игрока в гост чат из-за сей кода когда игрок находится в теле

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

Часть кода стала более читабильнее(наверно).
fixes #7829
fixes #4188
closes #4881

Авторство

Чеинжлог

🆑

  • add[link]: Новые эффекты артефактов
  • add: Сломанные мехи можно раскопать
  • fix: Говорящая маска не заставляет игрока говорить в гостчат
  • fix: Отчеты артефактов дают очки изучения

@TauKitty
Copy link
Contributor

TauKitty commented Sep 8, 2021

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Fix labels Sep 8, 2021
@Morar1 Morar1 changed the title xenoarcheology part code readdability, some artifact changes xenoarchaeology part code readability, some artifact changes Sep 8, 2021
/**
* Rebuilds proxy trigger zone, does this on a move
*/
/obj/machinery/artifact/proc/rebuild_zone()
Copy link
Contributor

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.

Это не с тг, а с алтаря капелланов. Но там зоны реже пересобираются

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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

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

Это не с тг, а с алтаря капелланов.

Да

Copy link
Contributor

@LudwigVonChesterfield LudwigVonChesterfield left a comment

Choose a reason for hiding this comment

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

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

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

Morar1 and others added 5 commits September 10, 2021 14:55
Co-authored-by: LudwigVonChesterfield <andreydell@yahoo.com>
removes random_unrestricted
creates list define for relic water
turfs orange to RANGE_TURFS
clear zone on move start
@Daoprotoss
Copy link

А можно еще в этом ПРе добавить бумажкам скана артефактов хоть какую-то научную ценность?..

…t working code for finds points

not going to implement /list/artifact_types giving extra points, it wasnt working anyway...
@Morar1 Morar1 marked this pull request as draft September 14, 2021 03:54
@Morar1 Morar1 marked this pull request as ready for review September 14, 2021 07:14
@LudwigVonChesterfield LudwigVonChesterfield merged commit a0f68ba into TauCetiStation:master Sep 14, 2021
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Sep 14, 2021
@Morar1 Morar1 deleted the xenoarchorsmth branch September 14, 2021 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Fix
Projects
None yet
5 participants