Zukero / andors-trail

Andor's Trail
135 stars 68 forks source link

Similar names with small difference #79

Closed urbalazs closed 6 years ago

urbalazs commented 6 years ago

Hello @Rijackson

I execute this command to extract the capitalized words (names expected): grep -ho '[A-Z][a-z][^" .,;?!*]*' AndorsTrail/res/raw/*.json | sort -u

After a manual checking I found these names, which are similar, but only one letter differs:

Can you please check what are the correct names?

Rijackson commented 6 years ago

Arghes (Remgard) and Arghest (BWM) are different NPCs Aryfola is a typo. It should be Aryfora. Good catch. I already fixed one of these, but evidently there is one more. This needs to be changed. Berbanes should be Berbane's. Another good catch. This also needs to be changed. Damerialia is a typo. It's only in a non-displayed quest though, so the users will never see it. There would be little benefit to changing this, but zero benefit to not changing it, so do as you wish. Damerilias is the plural of Damerilia, so that's fine. Elythara is a name, Elytharan means something pertaining to that name, so no problem. Erumem is the correct name, but erumen occurs only in internal IDs, so the player will never see it. It should be left as is, because the risk of breaking something by changing it far outweighs any benefit to the player (since such benefit would be zero).

You have not said how you want to be credited for this work in the game credits. We believe everyone that has contributed should get credit. It has been suggested within the dev team that the category should be "QA assistance", but how do you want your "name" to be displayed? urbalazs, Balázs Úr, Balázs Úr (urbalazs), something else?

urbalazs commented 6 years ago

Damerilias is the plural of Damerilia, so that's fine.

As I see, damerilias are "flowers that grow only in Remgard", right? We have both Damerilias (with capital letter) and damerilias (with lowercase letter). What is the correct? In my opinion it should be written with lowercase letters as other flower names are written: roses, daffodils, etc.

Erumem is the correct name, but erumen occurs only in internal IDs.

"Erumen" occurs not only in internal IDs, but in monsterlist_v0611_monsters1.json (4 times):

"Erumem" occurs 6 times in monsterlist_v0611_monsters1.json and monsterlist_v070_lodarmaze.json and doesn't occur in internal IDs:

The internal ID is "erumen". Are you sure that "erumem" is the correct name? It seems that it should be "erumen".

Zukero commented 6 years ago

Damerilias need a lowercase d indeed, except when used as the item name, or to begin a sentence.

Rijackson commented 6 years ago

Yes, you are right about Erumen and Erumem. The ID is erumen, so I guess we should switch all occurrences to that spelling. There are capitalization problems too. Hardened Erumen Lizard should be Hardened erumen Lizard.

Zukero commented 6 years ago

Why capitalize Lizard?

Rijackson commented 6 years ago

Sorry. Typo! Strong erumen lizard.

urbalazs commented 6 years ago

I found some other issues:

Rijackson commented 6 years ago

1) They should be consistent, and I think they should all be venomscale. Since they are inconsistent we are free to choose which one we prefer, and as you say, venomscales scales sounds weird.

2) Agreed.

3) It's there to allow the hero sprite to be placed on the top of the monument. There is some dialogue for that NPC: "$playername monument - not bad.". We don't want a name to show up there, because the hero's name is not known.

Zukero commented 6 years ago
  1. Agreed overall. I know at one point Halvor asks for venomscales' scales, meaning scales coming from several venomscales snakes. Maybe there's a better way to ask for these.

  2. Agreed.

  3. This would be better using a key area. A "dynamic placeholder" sprite will handle the graphical side of things.

urbalazs commented 6 years ago

Fixed with https://github.com/Zukero/andors-trail/commit/1c7f081f18e8b894e087fcfda3ad7483592c321b.