View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003021 | FSSCP | tables | public | 2014-03-17 03:02 | 2022-06-10 21:53 |
| Reporter | Spoon | Assigned To | DahBlount | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | assigned | Resolution | open | ||
| Product Version | 3.7.0 | ||||
| Summary | 0003021: "target requires fov" flag does not take $Turret Base FOV: in account | ||||
| Description | Causing turrets to remain idle instead of firing at an other valid target within range. | ||||
| Tags | No tags attached. | ||||
|
|
|
|
|
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? |
|
|
https://github.com/scp-fs2open/fs2open.github.com/pull/184 Pull Request |
|
|
Already a flag for this: "only target if can fire" |
|
|
"only target if can fire" is useless and doesn't do anything in this case. |
|
|
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. |
|
|
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. |
|
|
Might no longer be an issue. EatThePath to check with Warmachine. |
| 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 |
