Author Topic: Rename attack boosts  (Read 1077 times)

Omega

  • MegaGlest Team
  • Dragon
  • ********
  • Posts: 6,167
  • Professional bug writer
    • View Profile
    • Personal site
Rename attack boosts
« on: 22 July 2014, 01:44:11 »
I'd like to suggest that attack boosts be renamed. The name is inaccurate and very misleading. Until you read the documentation, it's not clear that it can boost stats besides attacks or be associated with skills other than attacks. If I recall correctly, the name is a hold-over from older times.

So I propose that we depreciate the `attack-boost` element (in the same way that `attack-strenhgt` (or however they spelled it) is depreciated). We can support both `attack-boost` and some new name (the XML parser has simple means of doing this, which are already used for `attack-strength`.

I think we should use the name `emanation` to describe attack boosts. This is the term that GAE used. MG's attack boosts and GAE's emanations are roughly the same thing. Dictionary-wise, emanation is also more accurate: "an abstract but perceptible thing that issues or originates from a source". Alternatives include "aura", "automatic-effect", or simply "boost" (using one of these would allow our terminology to be distinct from GAE -- I can think of pros and cons to this). "Boost" is what the English transformation is already using in-game.

If we do rename attack boosts, we should consider doing so in the source code, too (very easy to do with a bit of find and replace).
Edit the MegaGlest wiki: http://docs.megaglest.org/

My personal projects: http://github.com/KatrinaHoffert

Coldfusionstorm

  • Golem
  • ******
  • Posts: 868
    • View Profile
Re: Rename attack boosts
« Reply #1 on: 22 July 2014, 14:38:26 »
i would have to agree on this, This is a oddity that it's called like this in it's current form.

I vote yes  :thumbup: on a rename
WiP Game developer.
I do danish translations.
"i break stuff"

titi

  • MegaGlest Team
  • Airship
  • ********
  • Posts: 4,240
    • View Profile
    • http://www.titusgames.de
Re: Rename attack boosts
« Reply #2 on: 28 July 2014, 00:28:18 »
I personally don'tt like the "emanations" too much but I am no native english speaker. Another reason is that it is not the same as GAEs implementation and not really compatibel with this.  What about simply calling it "effect" or something like  "area effect"?

But if  its renamed it must be compatible with old things so new and old syntax works.

Try Megaglest! Improved Engine / New factions / New tilesets / New maps / New scenarios

MoLAoS

  • Ornithopter
  • *****
  • Posts: 433
    • View Profile
Re: Rename attack boosts
« Reply #3 on: 28 July 2014, 03:19:51 »
GAE also has effects. Emanations are effects on a unit rather than a skill. So you would still be using GAE terminology.

Emanation means to spread from a central point or source.

John.d.h

  • Moderator
  • Airship
  • ********
  • Posts: 3,757
  • I have to go now. My planet needs me.
    • View Profile
Re: Rename attack boosts
« Reply #4 on: 28 July 2014, 05:51:35 »
I think "aura" is better than "emanation", just for simplicity.  My issue with "boost" is that the word means an increase specifically, so it makes less sense if it's something that causes harm or weakness.  That may not be that big of a deal.  My favorite alternative would be "influence".

It may be worth noting that in GAE, emanations and effects are separate but related, with emanations just being a passive delivery system for effects.  Effects can also be applied by attacks.  I know there was a "cast-spell" skill in the works that would have worked for actively applying an effect without attacking (so a medic could heal an ally, for example), but I don't know if it was ever implemented.

MoLAoS

  • Ornithopter
  • *****
  • Posts: 433
    • View Profile
Re: Rename attack boosts
« Reply #5 on: 28 July 2014, 06:00:28 »
I think "aura" is better than "emanation", just for simplicity.  My issue with "boost" is that the word means an increase specifically, so it makes less sense if it's something that causes harm or weakness.  That may not be that big of a deal.  My favorite alternative would be "influence".

It may be worth noting that in GAE, emanations and effects are separate but related, with emanations just being a passive delivery system for effects.  Effects can also be applied by attacks.  I know there was a "cast-spell" skill in the works that would have worked for actively applying an effect without attacking (so a medic could heal an ally, for example), but I don't know if it was ever implemented.

It was not fully implemented. I looked at that and implemented some basic stuff while working on Mandate.

Aura is a much more common term for what GAE calls emanations in video game development.

titi

  • MegaGlest Team
  • Airship
  • ********
  • Posts: 4,240
    • View Profile
    • http://www.titusgames.de
Re: Rename attack boosts
« Reply #6 on: 28 July 2014, 11:53:04 »
I like Aura alot more than emanation, and for non native english speakers it is a much more common word.
Try Megaglest! Improved Engine / New factions / New tilesets / New maps / New scenarios

Omega

  • MegaGlest Team
  • Dragon
  • ********
  • Posts: 6,167
  • Professional bug writer
    • View Profile
    • Personal site
Re: Rename attack boosts
« Reply #7 on: 4 August 2014, 18:16:48 »
So, titi, softcoder, can we rename them? If desired, I can make this change.

What I would do is make "aura" (since that seems to be a term we can agree on) an alias to attack-boost (in the same way that attack-strenght and attack-strength are aliases). I'd then rename AttackBoost to Aura. Do a grep for various combinations of attack boost and rename them appropriately (so that the code is consistent in naming things).

Once the change is in a release, we can rename the term in our documentation and declare attack-boost as depreciated. As an aside, I'm thinking that the --validate-* functionalities should warn about depreciated elements (not sure if they already do).

Note that this will create conflicts between any awaiting changes that modify attack boosts. So my tags change would have to be integrated first (possibly others?).
Edit the MegaGlest wiki: http://docs.megaglest.org/

My personal projects: http://github.com/KatrinaHoffert

 

anything