View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001525 | FSSCP | physics | public | 2007-11-20 10:40 | 2012-01-21 04:39 |
| Reporter | Goober5000 | Assigned To | Valathil | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | no change required | ||
| Summary | 0001525: Three bugs reported by razorjack (having to do with phys_info.vel) | ||||
| Description | I've not looked at these closely; I'm just reporting them. Razorjack never added these into Mantis like I asked him too, so I'm doing it... Message 1: I've found a bug in (collidedebrisship.cpp, line 543) The vector of pship->phys_info.vel is referenced to world coordinate system. We need to translate it into object's local coordinate system before we are able to use its z component. Message 2: There are two similar bugs in aicode.cpp at line 8774: match_accel = target_objp->phys_info.vel.xyz.z / Ship_info[Ships[attack_objp->instance].ship_info_index].max_vel.xyz.z; and 8929: int moving = (En_objp->phys_info.vel.xyz.z > 0.5f); Message 3: I've searched through the source if there are similar accesses to components of vel and found 3 places in missionparse.cpp | ||||
| Tags | No tags attached. | ||||
|
|
Similar case (rotate/unrotate) issue seems to happen also elsewhere... For example in ai_turret.cpp vm_vec_rotate(&turret_norm, &turret_subsysp->system_info->turret_norm, &objp->orient); |
|
|
Are any of these issues still current with existing code? |
|
|
physics info velocity IS in local coordinates so the use of .xyz.z as the forward velocity is fine |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2007-11-20 10:40 | Goober5000 | New Issue | |
| 2009-01-16 23:01 | Wanderer | Note Added: 0010536 | |
| 2010-12-14 04:37 | Zacam | Note Added: 0012556 | |
| 2010-12-14 04:37 | Zacam | Status | new => feedback |
| 2011-06-24 04:42 | Zacam | Category | --------- => physics |
| 2012-01-21 04:39 | Valathil | Note Added: 0013088 | |
| 2012-01-21 04:39 | Valathil | Assigned To | => Valathil |
| 2012-01-21 04:39 | Valathil | Status | feedback => closed |
| 2012-01-21 04:39 | Valathil | Resolution | open => no change required |
