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
target_neighbor name #62
Comments
Any suggestions for this? The name target_neighbor seemed clear when I was implementing it. There's probably properties that have confusing names, so maybe documenting them in the wiki would be a better option. |
Maybe it's a language thing, and the secondary usage of "neighbor" as "nearby" or "adjacent" is not obvious to other languages? I'm open to better names. It's hard to come up with labels that are precise and short. But yes, with more documentation we could clear some of this up. |
So let me see if I have this right: if target_neighbor is set to 3, then the power will select a valid target (whether that's an enemy or a tile) within three tiles to target. Yes? I might go for target_area or target_within or even autotarget_within (or if I have the functionality slightly wrong, maybe random_within). |
It won't target enemies, only tiles. Whether or not a enemy is on the tile is irrelevant. |
When talking about areas, I'd use the word 'range' maybe? |
I think when I first was looking through mods, target_neighbor to ME said, "This boolean will determine whether this power targets 'self' or an 'adjacent enemy'", which was the source of my confusion. Though neighbor can be used flexibly where I live out in the boonies (I have 'neighbors' five houses away half a mile down the road), to me, the word means "people who live in property that abuts mine," so I think that's where I picked up the connotation that target_neighbor would be a "who" (not a "where") and a boolean, not a range. If we can't agree on a better name, then I agree with Clint that documentation might be the way to go. When 0.18 is comes out, I might try and do a Wiki writeup on "The Anatomy of a Power". |
Pennomi just told me what this does! It's not intuitive at all from the name. Maybe it should be renamed?
The text was updated successfully, but these errors were encountered: