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

Законы на ИИшки русском #201

Closed
wants to merge 4 commits into from

Conversation

Pik1ta
Copy link

@Pik1ta Pik1ta commented Feb 7, 2025

Первый пуллреквест, добейте меня.

Ну я чета головой ударился и сделал первый ПР. Вот.
@Pik1ta
Copy link
Author

Pik1ta commented Feb 7, 2025

Пикита забыл перевести один лавсет и подправить точки.

@Glamyrio
Copy link

Glamyrio commented Feb 7, 2025

@Artemchik542

Copy link
Collaborator

@Artemchik542 Artemchik542 left a comment

Choose a reason for hiding this comment

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

  • У тебя нет переопределения файла-фраз на твой.
  • Надобно теперь все новые подобные фичи-переводы заливать в модульную папку modular_meta в поддиректорию ru_translate

@@ -0,0 +1,499 @@
#define AI_LAWS_ASIMOV "asimov"
Copy link
Collaborator

Choose a reason for hiding this comment

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

тут вообще ничего не поменял, незачем такой файл вообще делать

Comment on lines +15 to +22
/datum/ai_laws/united_nations/add_inherent_law(law)
return //nuh uh

/datum/ai_laws/united_nations/add_ion_law(law)
return //nope!

/datum/ai_laws/united_nations/add_hacked_law(law)
return //nice try (emagging borgs still hard replaces this lawset though, and that's fine.)
Copy link
Collaborator

Choose a reason for hiding this comment

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

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

Comment on lines +125 to +137
message = "ТЕПЕРЬ [ionnumberbase] [ionnumbermodhalf][ionadjectiveshalf][ionthreats] НА СТАНЦИИ..."
if(2) //X is a crewmember/job
message = "ТЕПЕРЬ [ionnumberbase] [ionnumbermodhalf][ionadjectiveshalf][ioncrew1] НА СТАНЦИИ"
if(3) //X is an object
message = "ТЕПЕРЬ [ionnumberbase] [ionnumbermodhalf][ionadjectiveshalf][ionobjects] НА СТАНЦИИ"
if(4 to 6) //You can't call the shuttle because X is on the station
switch(rand(1,3)) //What is X?
if(1) //X is a threat
message = "ШАТТЛ НЕ МОЖЕТ БЫТЬ ВЫЗВАН, ПОТОМУ ЧТО [ionnumberbase] [ionnumbermodhalf][ionadjectiveshalf][ionthreats] НА СТАНЦИИ"
if(2) //X is a crewmember/job
message = "ШАТТЛ НЕ МОЖЕТ БЫТЬ ВЫЗВАН, ПОТОМУ ЧТО [ionnumberbase] [ionnumbermodhalf][ionadjectiveshalf][ioncrew1] НА СТАНЦИИ"
if(3) //X is an object
message = "ШАТТЛ НЕ МОЖЕТ БЫТЬ ВЫЗВАН, ПОТОМУ ЧТО [ionnumberbase] [ionnumbermodhalf][ionadjectiveshalf][ionobjects] НА СТАНЦИИ"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Вот это изменяешь прям в оригинальном файле, обязательно со стандартизированными пометками (а-ля "здесь был Вася"), про которые подробно рассказано в гайде про "Модульное ведение проекта"

Однако тут пока не трогай, тут очень много где надо менять... надобно подумать как с этим файлом поступить

@@ -0,0 +1,592 @@
/datum/round_event_control/ion_storm
Copy link
Collaborator

Choose a reason for hiding this comment

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

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


/datum/round_event/ion_storm/announce(fake)
if(prob(announce_chance) || fake)
priority_announce("Ion storm detected near the station. Please check all AI-controlled equipment for errors.", "Anomaly Alert", ANNOUNCER_IONSTORM)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Это пока менять не будет, когда вдруг будут переводы звуковых оповещений - тогда и это тоже поменяем

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants