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

Relics-related changes #1603

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Reiquard
Copy link
Contributor

Несколько раз в игре встречал реликвии с названиями, построенными по формуле [memeAdjective_masc] [thingLabel],
где в качестве второго аргумента были вещи: чаша, рукоять и скрижаль, что, понятно, приводило, например, к таким результатам — первый чаша, единый рукоять, свободный скрижаль.

Путей решения этой проблемы я видел два: приведение рода всех реликвий к мужскому или удаление вышеуказанной формулы из правил генерации. (Есть и третий — представить все реликвии в ж.р. или в ср.р, но это привело бы к более неуклюжим вариантам, чем в этом реквесте.)
Я пошёл по первому пути.

Вторая часть реквеста нацелена на изменения генерации названий по формуле [first][second].
Обе группы аргументов я разделил на две логические части:
first_prefixан, экс, гекс и пр. и secondзвено, локус, плеть и пр., соединяемые дефисом;
и second_nounвестник, глашатай и пр. и first_attrбездны, жизни и пр.
Добавил правило генерации [second_noun] [memeConcept] с небольшим шансом. (Если такая "самодеятельность" не приветствуется — дайте знать.)
Учёт и распределение весов произведены.

Comment on lines 69 to 71
<li>r_ideoRelicName(p=1.2)->[first][second]</li>
<li>r_ideoRelicName(p=0.5)->[second_noun] [first_attr]</li>
<li>r_ideoRelicName(p=0.3)->[second_noun] [memeConcept]</li>
Copy link
Contributor

Choose a reason for hiding this comment

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

В текущих условиях first_attr правильнее будет переименовать в generic_concept. Ибо это слово уже не в начале

<li>second->нос</li>
<li>second->вестник</li>
<li>second->лжец</li>
<li>first(p=15)->[first_prefix]-</li>
Copy link
Contributor

Choose a reason for hiding this comment

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

Оба варианта [first] теперь проще заинлайнить в месте вызова - так будет гораздо читаемее

    <li>r_ideoRelicName(p=0.4)->[memeHyphenPrefix][second]</li>
    <li>r_ideoRelicName(p=0.8)->[first_prefix]-[second]</li>

@xBLOKx
Copy link
Contributor

xBLOKx commented Apr 21, 2024

чаша, рукоять, скрижаль / потир, эфес, туппум

А стоит ли?

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

По крайней мере мне так кажется.

@Reiquard
Copy link
Contributor Author

чаша, рукоять, скрижаль / потир, эфес, туппум
Если другие слова ещё со скрипом ладно, то вот туппум весьма желательно чем то заменить.

@xBLOKx , прошу внести свои предложения.

@xBLOKx
Copy link
Contributor

xBLOKx commented Apr 21, 2024

@Reiquard , а вот предложений то у меня и нет к сожалению. Не знаю я подходящих слов да ещё, что бы склонялись они должным образом. Сказал бы, что тогда назвать хотя бы "глиняная табличка" но увы это всё тот же "свободный глиняная табличка". Может быть ещё вариант "свободный глиняный каталог" в какой-то мере имеет смысл, но всё же каталог звучит как-то не очень правильно хотя технически может им быть. Даже назвал бы "свиток" но увы это вроде не бумажный/пергаментный вариант. А доска, плита, табличка, писание отметаются всё по тем же причинам...

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

<li>r_ideoRelicName->[thingLabel] [memeConcept]</li>
<li>r_ideoRelicName->[memeAdjective_masc] [thingLabel]</li>
<li>r_ideoRelicName->{replace: [memeAdjective_masc] [thingLabel]; "ый Чаша"-"ая Чаша"; "ий Чаша"-"ая Чаша"; "ой Чаша"-"ая Чаша"; "ый Рукоять"-"ая Рукоять"; "ий Рукоять"-"ая Рукоять"; "ой Рукоять"-"ая Рукоять"; "ый Скрижаль"-"ая Скрижаль"; "ий Скрижаль"-"ая Скрижаль"; "ой Скрижаль"-"ая Скрижаль"}</li>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Для «полного» эффекта требуются изменения memeAdjective_masc в 1601.

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