Skip to content

- исправлены ключи в файлах правил генерации названий#1732

Open
Skiller87 wants to merge 7 commits into
Ludeon:masterfrom
Skiller87:v1.6.Odyssey-DefInjected-RulePackDef_part2
Open

- исправлены ключи в файлах правил генерации названий#1732
Skiller87 wants to merge 7 commits into
Ludeon:masterfrom
Skiller87:v1.6.Odyssey-DefInjected-RulePackDef_part2

Conversation

@Skiller87

Copy link
Copy Markdown
Contributor
  • спасатели -> утилизаторы
  • е -> ё
  • модифицированы правила генерации названий уникального оружия

- спасатели -> утилизаторы
- е -> ё
- модифицированы правила генерации названий уникального оружия

@Reiquard Reiquard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Замечание к реквесту в целом

Нужно актуализировать вероятности (веса́) практически во всех блоках правил.

<li>r_name->[spaceAdj_mas] [groupname_mas]</li>
<li>r_name->[spaceAdj_fem] [groupname_fem]</li>
<li>r_name->[spaceAdj_plural] [Enemies]</li>
<li>r_name->[Enemies] c [spaceNoun_gen]</li>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Возможные результаты генерации:

  • Чудовища c Звезды
  • Вампиры c Затмения

В первом варианте перед словоформой звезды требуется предлог со.
Второй вариант вообще едва ли допустим.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

  • предлоги вынес в spaceNoun_gen
  • вариант "Затмения" удалил.

Comment on lines 46 to 52
<li>r_name->[group] с [spaceNoun_gen]</li>
<li>r_name->[spaceAdj_neu] [Group_neutral]</li>
<li>r_name->[spaceAdj_masc] [Group_masc]</li>
<li>r_name->[spaceAdj_fem] [Group_fem]</li>
<li>r_name->[ColorBadass_plural] [spaceNoun_plural]</li>
<li>r_name->[ColorBadass_mas] [spaceNoun_mas]</li>
<li>r_name->[ColorBadass_masc] [spaceNoun_masc]</li>
<li>r_name->[ColorBadass_fem] [spaceNoun_fem]</li>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Выявить неработающие правила поможет схема с их одиночным включением перед проверкой:
Image

А для ускорения этого процесса, чтобы перед каждой такой проверкой не перезапускать игру, рекомендую установить мод Rim Language Hot Reload.

Так вот, эти правила не работают.

@Skiller87 Skiller87 Jun 27, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

не работало потому что переменная group и Enemies не были определены. Игра не включает в это рулпак нужных переменных и других рулпаков. Исправлено за счёт объявления этих переменных в рамках рул-пака.

<li>r_name(p=2)->[spaceNoun_fem] [NamePerson]</li>
<li>r_name(p=2)->[spaceNoun_masc] [NamePerson]</li>
<li>r_name(p=2)->[spaceNoun_neu] [NamePerson]</li>
<li>r_name(p=2)->[SpaceCollectiveNoun_fem] [NamePerson]</li>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
<li>r_name(p=2)->[SpaceCollectiveNoun_fem] [NamePerson]</li>
<li>r_name(p=2)->[spaceCollectiveNoun_fem] [NamePerson]</li>

 
Смену регистра первого символа нужно сделать и во всех остальных вхождениях переменных SpaceCollectiveNoun_*.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

поправил.

<li>SpaceCollectiveNoun_mas->Флот</li>
<li>SpaceCollectiveNoun_plural->Корсары</li>
<li>SpaceCollectiveNoun_plural->Пираты</li>
<li>SpaceCollectiveNoun_plural->Мародеры</li>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Мародеры -> Мародёры

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

поправил.

<li>r_name->[spaceAdj_fem] [groupname_fem]</li>
<li>r_name->[spaceAdj_plural] [Enemies]</li>
<li>r_name->[Enemies] c [spaceNoun_gen]</li>
<li>r_name->[group] с [spaceNoun_gen]</li>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

См. комментарий к строке 45.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

поправлено (см. первый коммент)

Comment on lines +129 to +131
<li>narrative_name->[Direction_masc] [thing_badass_masc]</li>
<li>narrative_name->[Direction_fem] [thing_badass_fem]</li>
<li>narrative_name->[Direction_neutral] [thing_badass_neu]</li>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Возможные результаты генерации:

  • Северо-Восточный Охотник На Гиен
  • Юго-Западное Страдание
  • Западная Саламандра

Сомневаюсь в уместности таких вариантов.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

удалил варианты с ключами Direction_*

<li>badass_adjective_fem->гибельная</li>
<li>badass_adjective_neu->гибельное</li>
<li>badass_adjective_mas->громовой</li>
<li>badass_adjective_mascc->громовой</li>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
<li>badass_adjective_mascc->громовой</li>
<li>badass_adjective_masc->громовой</li>

<li>weapon_adjective_fem->[badass_adjective_fem]</li>
<li>weapon_adjective_neu->[badass_adjective_neu]</li>
<li>badass_adjective_mas->мрачный</li>
<li>weapon_adjective(p=2)->[trait_adjective]</li>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

В переменной weapon_adjective нет необходимости, в соответствующем правиле следует использовать непосредственно trait_adjective.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Значимые переменные weapon_type и trait_adjective в текущем состоянии перевода встречаются при генерации с вероятностями ~0,21 и ~0,03 соответственно, что достаточно редко в сравнении с оригиналом, где вероятности их появления ~0,64 и ~0,42. Следует над этим поработать.

Также нужно провести работу по согласованию обозначенных переменных с правилами в этом файле.
Значения переменной weapon_type находятся здесь, а trait_adjectiveздесь.

<li>platform->dock</li>
<li>platform->ring</li>
-->
<NamerSettlementTradersGuild.rulePack.rulesStrings>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Считаю, имеет смысл представить в виде прилагательных те значения platAdj, которые возможно, чтобы результаты генерации были более естественными, например:

  • Станция Космоса -> Космическая Станция
  • Док Неба -> Небесный Док
  • Узел Галактики -> Галактический Узел
  • Платформа Звёзд -> Звёздная Платформа

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

реализовал для

космическая<
звёздная<
небесная<
галактическая<

@Skiller87 Skiller87 force-pushed the v1.6.Odyssey-DefInjected-RulePackDef_part2 branch from 16a415e to 32c983d Compare May 28, 2026 19:45
Skiller87 and others added 6 commits June 29, 2026 22:11
-предлоги "с" и "со" вынесены в spaceNoun_gen в правилах NamerFactionSalvagers;
-вариант "Затмения" удалил;
-исправлены неработающие правила из-за отсутствия доступа к ключам (group и Enemies);
-Мародеры -> Мародёры;
-SpaceCollectiveNoun_* -> -spaceCollectiveNoun_*;
-добавлены прилагательные platAdj_* в NamerSettlementTradersGuild для более естественных вариантов (например Станция Космоса -> Космическая Станция).

RulePacks_Namers_Gravship:
-удалил варианты с ключами Direction_*.
Co-authored-by: Николай Володин <62316596+Reiquard@users.noreply.github.com>
Co-authored-by: Николай Володин <62316596+Reiquard@users.noreply.github.com>
Co-authored-by: Николай Володин <62316596+Reiquard@users.noreply.github.com>
…амечания в реквесте

- удалено значение переменной ship_word_masc со значением где используется [AdjectiveBadass] во избежании дубликатов прилагательных при генерации
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.

2 participants