View Issue Details

IDProjectCategoryView StatusLast Update
0004471Spring engineGeneralpublic2015-03-24 12:19
ReporterSkasi Assigned Tocleanrock  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version97.0.1+git 
Fixed in Version98.0.1+git 
Summary0004471: Area commands are spherical instead of cylindrical
DescriptionArea commands such as arearepair and areareclaim only consider targets within a sphere. That means arearepair does not always target units flying at a high altitude even when they are above the area.
Steps To ReproduceHave a damaged gunship fly high above ground.
Issue a small area repair command right below the gunship. Make sure that the radius of the area command is smaller than the flight height.
TagsNo tags attached.
Checked infolog.txt for Errors

Activities

abma

2015-03-18 16:16

administrator   ~0014193

afaik repair range is spherical, too. imo thats just a visual "problem", the command shows just a cylindrical area when in real its spherical.

cleanrock

2015-03-22 08:23

reporter   ~0014202

https://github.com/spring/spring/commit/38e1e9bf9f14ec025effef6f042cafc5b11c575a

This hopefully don't have unwanted effects. Please test and reopen if needed.

Skasi

2015-03-22 19:38

reporter   ~0014211

Last edited: 2015-03-22 19:40

Arearepair works fine now. Areareclaim for debris is still spherical.

Looking through the code of the edited file https://github.com/spring/spring/blob/38e1e9bf9f14ec025effef6f042cafc5b11c575a/rts/Sim/Units/CommandAI/BuilderCAI.cpp maybe both occurrences of "GetFeaturesExact" (for area-reclaim and area-resurrection of features) and "GetUnits" (for what looks like area-capture) need a third argument "false" as well.

cleanrock

2015-03-24 12:19

reporter   ~0014222

https://github.com/spring/spring/commit/e8196534ef8a2500e0fc4e5f862542bcdbe886ae

I hope it got it all this time.

Issue History

Date Modified Username Field Change
2014-07-11 13:32 Skasi New Issue
2015-03-18 16:16 abma Note Added: 0014193
2015-03-22 08:23 cleanrock Note Added: 0014202
2015-03-22 08:23 cleanrock Status new => resolved
2015-03-22 08:23 cleanrock Fixed in Version => 98.0.1+git
2015-03-22 08:23 cleanrock Resolution open => fixed
2015-03-22 08:23 cleanrock Assigned To => cleanrock
2015-03-22 19:38 Skasi Note Added: 0014211
2015-03-22 19:38 Skasi Status resolved => feedback
2015-03-22 19:38 Skasi Resolution fixed => reopened
2015-03-22 19:40 Skasi Note Edited: 0014211
2015-03-24 12:19 cleanrock Note Added: 0014222
2015-03-24 12:19 cleanrock Status feedback => resolved
2015-03-24 12:19 cleanrock Resolution reopened => fixed