View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002024 | FSSCP | AI | public | 2009-11-10 04:41 | 2022-06-11 02:05 |
| Reporter | chief1983 | Assigned To | Goober5000 | ||
| Priority | high | Severity | minor | Reproducibility | N/A |
| Status | closed | Resolution | suspended | ||
| Product Version | 3.6.11 | ||||
| Summary | 0002024: Design tweaks and refactoring in Sushi's AI code | ||||
| Description | It needs to be cleaned up. Per Goober. | ||||
| Tags | No tags attached. | ||||
|
|
More specific details/suggestions on what needs to be cleaned up and how would be uber-handy. I have "I wrote it" myopia. :) |
|
|
Goober please provide Sushi with the feedback that he requested. |
|
|
Closing this. Mantis is IMHO not the right venue for this kind of discussion; and simple one-liners like "a bit of a mess" are not constructive criticism. |
|
|
Reopening. There are a bunch of legitimate design concerns that should be addressed here, but I need to sit down and outline them. There should be a thread on HLP where we've discussed some of them, such as putting all the AI arrays under the umbrella of a struct rather than as separate instances. I'll provide Sushi with the requested feedback, but not until after 3.7 is out, since this is more of a design and future-proof ticket than a bugfix ticket. I've renamed the ticket since the original name was of course not really descriptive or constructive. |
|
|
I have unset the target version because there is no clearly defined bug in the description. |
|
|
This should never have been reopened. When you see a specific flaw, write an issue about that flaw. Reopening this so that it can stay open for another 10 years didn't solve anything. |
|
|
Moving to github: https://github.com/scp-fs2open/fs2open.github.com/issues/4382 |
|
|
> This should never have been reopened. When you see a specific flaw, write an issue about that flaw. Reopening this so that it can stay open for another 10 years didn't solve anything. There were several specific critiques in the thread that prompted this ticket. I should have linked that thread at the time I opened it, but I wanted to refine the critiques before doing so. I'll try to find that thread and link it on GitHub. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2009-11-10 04:41 | chief1983 | New Issue | |
| 2009-11-10 17:33 | Zacam | Status | new => assigned |
| 2009-11-10 17:33 | Zacam | Assigned To | => Sushi_CW |
| 2009-11-14 15:10 | Sushi_CW | Note Added: 0011281 | |
| 2010-08-05 20:07 | chief1983 | Target Version | 3.6.12 RC1 => 3.7.2 |
| 2012-10-28 00:57 | iss_mneur | Note Added: 0013996 | |
| 2012-10-28 00:57 | iss_mneur | Assigned To | Sushi_CW => Goober5000 |
| 2012-10-28 00:57 | iss_mneur | Status | assigned => feedback |
| 2012-10-28 00:57 | iss_mneur | Target Version | 3.7.2 => 3.7 |
| 2012-11-26 11:35 | The_E | Note Added: 0014191 | |
| 2012-11-26 11:36 | The_E | Status | feedback => closed |
| 2012-11-26 11:36 | The_E | Resolution | open => suspended |
| 2012-11-27 00:03 | Goober5000 | Note Added: 0014192 | |
| 2012-11-27 00:03 | Goober5000 | Status | closed => assigned |
| 2012-11-27 00:03 | Goober5000 | Resolution | suspended => open |
| 2012-11-27 00:03 | Goober5000 | Category | tables => AI |
| 2012-11-27 00:03 | Goober5000 | Target Version | 3.7 => |
| 2012-11-27 00:03 | Goober5000 | Summary | Sushi's AI stuff is a bit of an organizational mess => Design tweaks and refactoring in Sushi's AI code |
| 2012-11-27 00:03 | Goober5000 | Target Version | => 3.7.2 |
| 2012-11-27 00:04 | Goober5000 | Note Edited: 0014192 | |
| 2015-04-16 04:22 | Goober5000 | Target Version | 3.7.2 => 3.7.4 |
| 2015-09-22 21:29 | m_m | Note Added: 0016785 | |
| 2015-09-22 21:29 | m_m | Target Version | 3.7.4 => |
| 2022-06-10 16:47 | FSCyborg | Note Added: 0017114 | |
| 2022-06-10 16:48 | z64555 | Note Added: 0017115 | |
| 2022-06-10 16:49 | z64555 | Status | assigned => closed |
| 2022-06-10 16:49 | z64555 | Resolution | open => suspended |
| 2022-06-11 02:05 | Goober5000 | Note Added: 0017143 |
