View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0001778 | FSSCP | gameplay | public | 2008-09-26 09:28 | 2008-10-05 18:00 | ||||
Reporter | KeldorKatarn | ||||||||
Assigned To | |||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | resolved | Resolution | fixed | ||||||
Product Version | 3.6.9 | ||||||||
Target Version | Fixed in Version | 3.6.10 | |||||||
Summary | 0001778: ai_link_primaries problem | ||||||||
Description | The $AI Always Links Energy Weapons and $AI Maybe Links Energy Weapons AI Profile settings are not interpreted correctly: if (shipp->weapon_energy > The_mission.ai_profile->link_energy_levels_always[Game_skill_level]) { comparing the ship's absolute energy level to a percentage doesn't do much good, so this must be: if (shipp->weapon_energy * 100.0f / sip->max_weapon_reserve > The_mission.ai_profile->link_energy_levels_always[Game_skill_level]) { | ||||||||
Additional Information | Fix attached | ||||||||
Tags | No tags attached. | ||||||||
Attached Files |
|
![]() |
|
Goober5000 (administrator) 2008-09-27 01:54 |
Good catch. Add an ai_profiles flag and I can commit this patch. |
KeldorKatarn (reporter) 2008-09-27 07:54 |
I don't understand.. this is not retail. This was added by someone in FS_Open and uses above AI Profile lines to get the values. Why does this need an additional ai_profiles flag? It was a bug, this doesn't add a feature. The original feature was buggy. |
Goober5000 (administrator) 2008-09-27 20:28 |
This *is* a retail feature. Go back and read the code more carefully. Compare it to the original source code release. The only difference between retail and SCP is that in retail the percentages were defined in the code, and in SCP the percentages are defined in ai_profiles so that they can be edited. |
KeldorKatarn (reporter) 2008-09-27 22:59 Last edited: 2008-09-27 23:28 |
I see, so this bug was already in retail. I will make this a flag then alright. Edit: New patch attached. Feel free to change the comment line again to something more appropriate if you think that is needed. |
KeldorKatarn (reporter) 2008-09-29 10:38 |
Is this ok now? |
Goober5000 (administrator) 2008-09-30 20:44 |
Mostly. I tweaked it, then committed. |
KeldorKatarn (reporter) 2008-10-01 11:46 Last edited: 2008-10-01 11:48 |
Note withdrawn. Missread |
![]() |
|||
Date Modified | Username | Field | Change |
---|---|---|---|
2008-09-26 09:28 | KeldorKatarn | New Issue | |
2008-09-26 09:28 | KeldorKatarn | File Added: ai_link_primaries_fix.patch | |
2008-09-27 01:54 | Goober5000 | Note Added: 0009711 | |
2008-09-27 01:57 | Goober5000 | Status | new => confirmed |
2008-09-27 07:54 | KeldorKatarn | Note Added: 0009718 | |
2008-09-27 20:28 | Goober5000 | Note Added: 0009724 | |
2008-09-27 22:59 | KeldorKatarn | Note Added: 0009728 | |
2008-09-27 23:27 | KeldorKatarn | File Added: ai_link_primaries_fix_new.patch | |
2008-09-27 23:28 | KeldorKatarn | Note Edited: 0009728 | |
2008-09-29 10:38 | KeldorKatarn | Note Added: 0009740 | |
2008-09-30 20:44 | Goober5000 | Note Added: 0009752 | |
2008-09-30 20:44 | Goober5000 | Status | confirmed => resolved |
2008-09-30 20:44 | Goober5000 | Resolution | open => fixed |
2008-09-30 20:44 | Goober5000 | Fixed in Version | => 3.6.10 |
2008-10-01 11:23 | KeldorKatarn | Status | resolved => feedback |
2008-10-01 11:23 | KeldorKatarn | Resolution | fixed => reopened |
2008-10-01 11:46 | KeldorKatarn | Note Added: 0009757 | |
2008-10-01 11:48 | KeldorKatarn | Note Edited: 0009757 | |
2008-10-05 18:00 | Goober5000 | Status | feedback => resolved |
2008-10-05 18:00 | Goober5000 | Resolution | reopened => fixed |