1.6 fixed dependency logic error and added missing keys#1709
1.6 fixed dependency logic error and added missing keys#1709Skiller87 wants to merge 18 commits intoLudeon:masterfrom
Conversation
| <!-- <li>r_name->[terrain_word] [narrative_name] </li> | ||
| <li>r_name(p=0.05)->[terrain_word] [storyname] </li> --> | ||
| <!-- <li>narrative_name(p=12)->[WordTribal]</li> --> | ||
| <!-- <li>narrative_name(p=12)->[PlaceOutlander]</li> --> | ||
| <!-- <li>narrative_name->[Color][NaturalObject]</li> --> | ||
| <!-- <li>narrative_name->[AdjectiveNatural][BodyPart]</li> --> |
There was a problem hiding this comment.
Почему строки ниже - закомменчены?
There was a problem hiding this comment.
копировал с оригинала, не определился пока внедрять ли конструкции эти в рабочие правила, поэтому пока оставил. Этот реквест лучше отменить, в виду того , что я дальше проработал зависимости и текущие изменения не так актуальны.
There was a problem hiding this comment.
Ты мог бы залить коммиты с переработанными зависимостями сюда. И не придётся ничего отменять
There was a problem hiding this comment.
актуальный файл загружался с коммитом "fix and modification logic RulePackDef"
я не хотел бы удалять закомментированные строки. Есть ощущение что можно будет доработать эту часть чтобы генерации названий на карте.
Core/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Core/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
| <li>r_name(p=0.4)->Яд от [NamePerson]</li> | ||
| <li>r_name(p=0.4)->Порча от [NamePerson]</li> | ||
| <li>r_name(p=0.4)->Свалка в [NamePerson]</li> | ||
| <li>r_name(p=0.4)->[terrain_word] [narrative_name]</li> |
There was a problem hiding this comment.
В чём причина замены вероятности?
There was a problem hiding this comment.
не совсем понял, про какую замену речь?
сейчас в актуальном файле:
последняя фраза кажется более естественной , к примеру - "токсичное озеро Басто" естественнее чем "Яд от Jane"
There was a problem hiding this comment.
В оригинальном тексте вероятности другие, 0.05
<li>r_name(p=0.05)->[NamePerson]'s poison</li>
<li>r_name(p=0.05)->[NamePerson]'s blight</li>
<li>r_name(p=0.05)->[NamePerson]'s dumping grounds</li>
А у тебя 0.4. Почему?
There was a problem hiding this comment.
выложил коммит, в котором коэффициенты скорректировал в соответствии с оригиналом на сколько это возможно.
1087354 to
9ce6c17
Compare
b0c1af4 to
9ce6c17
Compare
There was a problem hiding this comment.
Резюмирующие замечания
При использовании уже имеющегося генератора названий населённых пунктов townname_wordgen, а я настоятельно рекомендую использовать именно его, следующие файлы становятся неактуальными и подлежат удалению:
- End_fem.txt
- End_mas.txt
- End_neo.txt
- End_plural.txt
В блоке WorldFeatureNamerCommon, где определяются переменные narrative_name_*, необходимо исключить варианты с частными значениями (forest_*, wasteland_*).
Аффиксы _neo в названиях переменных в рамках реквеста следует заменить на _neu.
Закомментированные строки, а именно отключённые правила генерации, следует удалить. В таком виде их можно оставить у себя в форке для дальнейшей работы, а в ветке master им не место.
Заключение
Учитывая большой объём требуемых исправлений и, по-видимому, продолжающуюся работу над некоторыми правилами, а также важность скорейшего исправления ошибок в игре, предлагаю разделить реквест на две части: в текущей части оставить все необходимые изменения, касающиеся непосредственно исправления отображающихся в игре ошибок, а с новым реквестом добавить недостающие и исправленные правила генерации.
Core/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Core/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_UniqueWeapons.xml
Outdated
Show resolved
Hide resolved
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_UniqueWeapons.xml
Outdated
Show resolved
Hide resolved
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Core/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Считаю важным обозначить, что текущая структура географических генераторов в локализации разительно отличается от оригинальной и потому вызывает затруднения в адаптации перевода новых дополнений. В этой связи принято решение изменить эту структуру, что, следовательно, в большей степени сделает работу по
Поэтому предлагаю не тратить силы на эту самую доработку, а учесть текущие замечания по реквесту, чтобы изменения быстрее оказались в мастере и в игре перестали появляться ошибки. |
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_UniqueWeapons.xml
Outdated
Show resolved
Hide resolved
я закончил, на сколько можно было без переделки структуры. |
Reiquard
left a comment
There was a problem hiding this comment.
Прошу быть внимательнее к таким, на первый взгляд, неприметным и незначительным изменениям, как добавление или удаление пустых строк и пробелов, а также смещение отступов. Несмотря на то что на работоспособность и функциональность локализации это не влияет, такие изменения ложатся дополнительной нагрузкой при разборе непереведённых строк после обновления игры.
Core/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Core/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
| <li>r_name->[spaceAdj_plural] [Enemy_plural]</li> | ||
| <li>r_name->[Enemy_plural] c [spaceNoun_gen]</li> | ||
| <li>r_name->[groupname] с [spaceNoun_gen]</li> | ||
| <li>r_name->[group] с [spaceNoun_gen]</li> |
There was a problem hiding this comment.
Проверялся ли в игре этот вариант?
| <li>r_name->[group] с [spaceNoun_gen]</li> | ||
| <li>r_name->[spaceAdj_plural] [groupname_plural]</li> | ||
| <li>r_name->[spaceAdj_mas] [groupname_mas]</li> | ||
| <li>r_name->[spaceAdj_masc] [groupname_masc]</li> |
There was a problem hiding this comment.
Переменная groupname_masc не определена.
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
Odyssey/DefInjected/RulePackDef/RulePacks_Namers_WorldFeatures.xml
Outdated
Show resolved
Hide resolved
-добавлены ключи narrative_name в Core (возможно требуют корректировки) - скорректированы и дополнены зависимости в файлах Odyssey для перевода обозначений объектов на карте (ранее перевод не работал)
- исправлены отступы - переписаны terrain_world с учётом дифференциации по роду и числу (в связи с этим количество правил увеличилось) . - добавлена логика генерации городов с учётом дифференциации по роду и числу для меток заражённых поселений добавлены текстовые файлы в core - исправлены косячные строки
проинспектированы вероятности в соответствии с оригиналом, на сколько это возможно. В оригинале генерация куда проще, соответственно не все вероятности совпадают.
- добавлены определения для ключей trait_adjective_mas, trait_adjective_fem, trait_adjective_neu - удалены ключи weapon_type_mas, weapon_type_fem и weapon_type_neu (видимо были ошибочно добавлены студией при создании перевода при выпуске Одисеи)
….xml Co-authored-by: Николай Володин <62316596+Reiquard@users.noreply.github.com>
Co-authored-by: Николай Володин <62316596+Reiquard@users.noreply.github.com>
….xml Co-authored-by: Николай Володин <62316596+Reiquard@users.noreply.github.com>
… на сколько было возможно. - ключи mas заменены на ключи masc - ключи neo заменены на neu - исправлена логика генерации названий уникального вооружения. - скорректированые названия файлов с учётом стандарта обозначения рода (*_neutral, *_masc)
- поправлены файлы по замечаниям, которые раннее откатились - логика генерации для названий городов упразднена до штатной - заменены некоторые слова на более подходящие по смыслу - исправлены ключи для названий фракций
…родов - удалены объявления ключей из файлов (окончания городов по родам) - упраздена логика - удалена строчка <li>story_result->disparate</li> (ошибочно попавшая в локализацию)
…ны отступы в соответствии с исходным видом.
- заменены ключи на рабочие в Namers_Factions - заменены ключи на рабочие в Namers_Gravship - убраны лишние ключи в Namers_UniqueWeapons p.s. работа на заключительной стадии в файле Namers_Landmarks (пока не выложил)
- заменены слова в Namers_Landmarks некоторых блоков (чтобы слова не пересекались между типами ориентиров) - обновлены вероятности (так как в оригинале подбор вариантов работает по другому, ориентировался по предоставленной инструкции, а в отдельных случаях сравнивал кол-ва результатов тех или иных названий с оригиналом чтобы добиться максимального совпадения)
…лов возвращены к мастеру, поправлены опечатки ключей, формулы генерации приведены в соответствии с замечаниями, вероятность скорректированы с учётом особенностей локализации)
- fix рода ключей (после повторной проверки)
…acks_Namers_Gravship) -replace probabilities (RulePacks_Namers_Landmarks)
e21f57a to
f6e70b6
Compare
-добавлены ключи narrative_name в Core (возможно требуют корректировки)