2019-10-19 11:58 EDT


View Issue Details Jump to Notes ]
IDProjectCategoryView StatusLast Update
0003021FSSCPtablespublic2017-01-20 14:35
ReporterSpoon 
Assigned ToDahBlount 
PrioritynormalSeverityminorReproducibilityalways
StatusassignedResolutionopen 
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 Files

-Relationships
+Relationships

-Notes

~0016747

DahBlount (developer)

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 (developer)

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

~0016749

FUBAR-BDHR (developer)

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

~0016750

Spoon (reporter)

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

~0016752

FUBAR-BDHR (developer)

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 (developer)

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.
+Notes

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