[86.0] maxAngleDif effectively doubled
Moderator: Moderators
-
- Moderator
- Posts: 2464
- Joined: 12 Oct 2007, 09:24
[86.0] maxAngleDif effectively doubled
In 85.0 maxAngleDif is the angle between the extremes of a weapon's arc. In 86.0 maxAngleDif is the maximum angle away from mainDir that a weapon can aim at. Effectively to update all values must be halved.
I added a unitdefs_posts fix in ZK as I'm waiting to see if the change is accidental or not. I didn't hear anything about it.
I added a unitdefs_posts fix in ZK as I'm waiting to see if the change is accidental or not. I didn't hear anything about it.
Re: [86.0] maxAngleDif effectively doubled
Now your gunships will jam again.
-
- Moderator
- Posts: 2464
- Joined: 12 Oct 2007, 09:24
Re: [86.0] maxAngleDif effectively doubled
So... explain what is going on. If I had to take a wild guess it would be that you effectively doubled all maxAngleDifs instead of telling us to increase the maxAngleDif for gunships that jam.
I tested this and my unitdefs_post change does not produce gunship weapon jamming if I also double the maxAngleDif of the jammable gunships.
I tested this and my unitdefs_post change does not produce gunship weapon jamming if I also double the maxAngleDif of the jammable gunships.
Re: [86.0] maxAngleDif effectively doubled
Thanks for the heads up, ill go half the angles for notair.
Re: [86.0] maxAngleDif effectively doubled
The variable is called "maxMainDirAngleDif" internally. That implies it is not supposed to be "the angle between the extremes of a weapon's arc" (as in 85.0) but exactly "the maximum angle away from mainDir", which is how 86.0 interprets it during targetting checks.
-
- Moderator
- Posts: 2464
- Joined: 12 Oct 2007, 09:24
Re: [86.0] maxAngleDif effectively doubled
I don't really care how the value is interpreted as long as no functionality is lost. The functionality is unchanged and I think the new interpretation makes a bit more sense, that is how I assumed it worked when I first encountered it. So this is a notification for moddevs and a check to see if the change was intentional.
Doubled gunship maxAngleDif fixes jamming in 86.0 which may be important to others as well.
Doubled gunship maxAngleDif fixes jamming in 86.0 which may be important to others as well.
Re: [86.0] maxAngleDif effectively doubled
When did this change? Seems I documented it this way for the Unitdefs wiki page, oddly - seen as I can't find any branch that doesn't default to 360 .
TBH it seems going all the way back
http://springrts.com/phpbb/viewtopic.php?f=12&t=4012
I prefer 'I want a 90 degree arc, I set the value to 90'. It's the way S44's arcs have been set since at least 2008. Why change it after 4-6 years?
If the internal name is a big deal, surely it makes more sense to change that and not break every single game that uses it, and a history of documentation (However muddled it is - I will correct the UnitDefs page).
TBH it seems going all the way back
http://springrts.com/phpbb/viewtopic.php?f=12&t=4012
Gnomre wrote:The cone is defined with "MaxAngleDif1=Degrees;" so if you wanted the weapon to only fire in a 60 degree arc, you'd put MaxAngleDif1=60;
zwzsg wrote:A 90 degree forward cone would look like:
WeaponMainDir1=0 0 1;//x:0 y:0 z:1 => that's forward!
MaxAngleDif1=90;//90° from side to side, or 45° from center to each direction
there are conflicting interpretations of how it works (Though I'd take gnome, and zwzsg's word any day)Das Bruce wrote:Firstly thats 45 degrees either way, cut it in half.
I prefer 'I want a 90 degree arc, I set the value to 90'. It's the way S44's arcs have been set since at least 2008. Why change it after 4-6 years?
If the internal name is a big deal, surely it makes more sense to change that and not break every single game that uses it, and a history of documentation (However muddled it is - I will correct the UnitDefs page).
Re: [86.0] maxAngleDif effectively doubled
Have you no respect for our customs? It's not a proper Spring release if it doesn't introduce a change breaking every single game! The more subtle the bug, the merrier.FLOZi wrote:If the internal name is a big deal, surely it makes more sense to change that and not break every single game that uses it, and a history of documentation (However muddled it is - I will correct the UnitDefs page).
Re: [86.0] maxAngleDif effectively doubled
wait, so my firearcs need to be adjusted? or it has been completely broken? I don't want to overreact but I really don't understand the change NOR will I have time to work on it. Sorry spring but my turtle is more important.FLOZi wrote:Gnomre wrote:The cone is defined with "MaxAngleDif1=Degrees;" so if you wanted the weapon to only fire in a 60 degree arc, you'd put MaxAngleDif1=60;zwzsg wrote:A 90 degree forward cone would look like:
WeaponMainDir1=0 0 1;//x:0 y:0 z:1 => that's forward!
MaxAngleDif1=90;//90° from side to side, or 45° from center to each directionthere are conflicting interpretations of how it works (Though I'd take gnome, and zwzsg's word any day)Das Bruce wrote:Firstly thats 45 degrees either way, cut it in half.
I prefer 'I want a 90 degree arc, I set the value to 90'. It's the way S44's arcs have been set since at least 2008. Why change it after 4-6 years?
If the internal name is a big deal, surely it makes more sense to change that and not break every single game that uses it, and a history of documentation (However muddled it is - I will correct the UnitDefs page).
Re: [86.0] maxAngleDif effectively doubled
85 is the same way it has always been.
Google reports that (current) 86.0 has doubled arcs in comparison.
Help me lobby-lobby-lobby to have the change reverted.
Lobby lobby badger badger rabble rabble.
Google reports that (current) 86.0 has doubled arcs in comparison.
Help me lobby-lobby-lobby to have the change reverted.
Lobby lobby badger badger rabble rabble.
Re: [86.0] maxAngleDif effectively doubled
+1 on having that change reverted.
- KingRaptor
- Zero-K Developer
- Posts: 838
- Joined: 14 Mar 2007, 03:44
Re: [86.0] maxAngleDif effectively doubled
I think it makes more sense the new way, and as far as breaking changes go it could have been a lot worse (this one can be fixed with 3 lines in unitdefs_post).
Documenting the change would have been nice though.
Documenting the change would have been nice though.
- SanadaUjiosan
- Conflict Terra Developer
- Posts: 907
- Joined: 21 Jan 2010, 06:21
Re: [86.0] maxAngleDif effectively doubled
I vote to keep things as they were.
I personally loved that I could enter 180 and I knew exactly what it meant. So many things in Spring require little calculations to figure out what they really represent, it was nice to have something that wasn't like that
I personally loved that I could enter 180 and I knew exactly what it meant. So many things in Spring require little calculations to figure out what they really represent, it was nice to have something that wasn't like that
Re: [86.0] maxAngleDif effectively doubled
I think it makes less sense. also, this would be fixing it with a kludge.KingRaptor wrote:I think it makes more sense the new way, and as far as breaking changes go it could have been a lot worse (this one can be fixed with 3 lines in unitdefs_post).
Is there an advantage to the change?
- Forboding Angel
- Evolution RTS Developer
- Posts: 14673
- Joined: 17 Nov 2005, 02:43
Re: [86.0] maxAngleDif effectively doubled
+1 on your +1hoijui wrote:+1 on having that change reverted.
-
- Moderator
- Posts: 2464
- Joined: 12 Oct 2007, 09:24
Re: [86.0] maxAngleDif effectively doubled
I couldn't care less and would prefer fixes for things that cannot be simply worked around. Such as the shield graphics.
Re: [86.0] maxAngleDif effectively doubled
Good. If the internal name doesn't reflect the external behaviour then the name has to be changed, not the behaviour.FLOZi wrote:https://github.com/spring/spring/commit ... 55b0984ade
restore old weapon targetting-arcs