Source Code Project Mantis - FSSCP
View Issue Details
0003021FSSCPtablespublic2014-03-16 23:022017-01-20 14:35
ReporterSpoon 
Assigned ToDahBlount 
PrioritynormalSeverityminorReproducibilityalways
StatusassignedResolutionopen 
PlatformOSOS Version
Product Version3.7.0 
Target VersionFixed in Version 
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.
Attached Filespng basefov.png (424,955) 2014-03-16 23:40
http://scp.indiegames.us/mantis/file_download.php?file_id=2337&type=bug

Notes
(0016747)
DahBlount   
2015-06-14 14:57   
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?
(0016748)
DahBlount   
2015-06-14 19:16   
https://github.com/scp-fs2open/fs2open.github.com/pull/184 Pull Request
(0016749)
FUBAR-BDHR   
2015-06-14 22:06   
Already a flag for this: "only target if can fire"
(0016750)
Spoon   
2015-06-14 22:26   
"only target if can fire" is useless and doesn't do anything in this case.
(0016752)
FUBAR-BDHR   
2015-06-14 23:50   
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.
(0016753)
DahBlount   
2015-06-16 14:14   
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.

Issue History
2014-03-16 23:02SpoonNew Issue
2014-03-16 23:40SpoonFile Added: basefov.png
2015-06-14 14:57DahBlountNote Added: 0016747
2015-06-14 19:16DahBlountNote Added: 0016748
2015-06-14 22:06FUBAR-BDHRNote Added: 0016749
2015-06-14 22:26SpoonNote Added: 0016750
2015-06-14 23:50FUBAR-BDHRNote Added: 0016752
2015-06-16 14:14DahBlountNote Added: 0016753
2017-01-20 14:35DahBlountAssigned To => DahBlount
2017-01-20 14:35DahBlountStatusnew => assigned