View Issue Details

IDProjectCategoryView StatusLast Update
0003021FSSCPtablespublic2022-06-10 21:53
ReporterSpoon Assigned ToDahBlount  
PrioritynormalSeverityminorReproducibilityalways
Status assignedResolutionopen 
Product Version3.7.0 
Summary0003021: "target requires fov" flag does not take $Turret Base FOV: in account
DescriptionCausing turrets to remain idle instead of firing at an other valid target within range.
TagsNo tags attached.

Activities

Spoon

2014-03-17 03:40

reporter  

basefov.png (424,955 bytes)

DahBlount

2015-06-14 18:57

administrator   ~0016747

I did some searching and perhaps adding a flag that makes a turret ignore the target radius would be a good solution here?

The flag would most likely take effect somewhere around aiturret.cpp, line 1448.

Anyone else have an opinion on this?

DahBlount

2015-06-14 23:16

administrator   ~0016748

https://github.com/scp-fs2open/fs2open.github.com/pull/184 Pull Request

FUBAR-BDHR

2015-06-15 02:06

developer   ~0016749

Already a flag for this: "only target if can fire"

Spoon

2015-06-15 02:26

reporter   ~0016750

"only target if can fire" is useless and doesn't do anything in this case.

FUBAR-BDHR

2015-06-15 03:50

developer   ~0016752

Well maybe there is a bug in the existing flag because that flag along with the advanced fov edge checks were added for exactly this purpose. The were put in so the Sobek turrets would pick a new target if the existing target went out of the fov.

DahBlount

2015-06-16 18:14

administrator   ~0016753

From looking at the code, it seems that "target requires fov" ultimately results in the SSF_FOV_REQUIRED flag being. The flag works as intended except that it does not make the turret switch targets if its current target passes out of the turrets FOV.

MjnMixael

2022-06-10 21:53

manager   ~0017142

Might no longer be an issue. EatThePath to check with Warmachine.

Issue History

Date Modified Username Field Change
2014-03-17 03:02 Spoon New Issue
2014-03-17 03:40 Spoon File Added: basefov.png
2015-06-14 18:57 DahBlount Note Added: 0016747
2015-06-14 23:16 DahBlount Note Added: 0016748
2015-06-15 02:06 FUBAR-BDHR Note Added: 0016749
2015-06-15 02:26 Spoon Note Added: 0016750
2015-06-15 03:50 FUBAR-BDHR Note Added: 0016752
2015-06-16 18:14 DahBlount Note Added: 0016753
2017-01-20 19:35 DahBlount Assigned To => DahBlount
2017-01-20 19:35 DahBlount Status new => assigned
2022-06-10 21:53 MjnMixael Note Added: 0017142