MantisBT - opensim
View Issue Details
0007226opensim[REGION] Physics Enginespublic2014-06-20 14:002015-08-18 09:34
vegaslon 
Robert Adams 
normalminoralways
closedfixed 
master (dev code) 
master (dev code) 
Standalone (1 Region) , Standalone (Multiple Regions) , Grid (1 Region per Sim) , Grid (Multiple Regions per Sim)
BulletSim
Unknown
None
0007226: [PATCH]Bulletsim-- Implement Vehicle Reference Frame.
This has not been working before but this small tweak allows you to use it for vehicles.
No tags attached.
patch BulletSim-Implement-hooks-for-VEHICLE_REFERENCE_FRAM.patch (9,366) 2014-07-19 09:38
http://opensimulator.org/mantis/file_download.php?file_id=3959&type=bug
Issue History
2014-06-20 14:00vegaslonNew Issue
2014-06-20 14:00vegaslonFile Added: 0001-BulletSim-Implement-working-VEHICLE_REFERENCE_FRAME.patch
2014-06-20 14:00vegaslonStatusnew => patch included
2014-07-18 19:00Robert AdamsNote Added: 0026511
2014-07-19 09:37vegaslonFile Deleted: 0001-BulletSim-Implement-working-VEHICLE_REFERENCE_FRAME.patch
2014-07-19 09:38vegaslonFile Added: BulletSim-Implement-hooks-for-VEHICLE_REFERENCE_FRAM.patch
2014-07-19 09:39vegaslonNote Added: 0026515
2014-07-26 16:11Robert AdamsAssigned To => Robert Adams
2014-07-26 16:11Robert AdamsStatuspatch included => assigned
2015-04-06 15:57vegaslonStatusassigned => resolved
2015-04-06 15:57vegaslonFixed in Version => master (dev code)
2015-04-06 15:57vegaslonResolutionopen => fixed
2015-08-18 09:34aiaustinNote Added: 0029242
2015-08-18 09:34aiaustinStatusresolved => closed

Notes
(0026511)
Robert Adams   
2014-07-18 19:00   
Reference frame would be cool to have but this is not the place to put this code. The VehicleOrientation method is a wrapper for the orientation in the underlying prim and does the caching of the value so the underlying prim is not updated multiple times (and causing all the side effects thereof). Changing the returned value here without also changing the setting would mess up orientation getting and setting.

For reference frame, displaced orientation there should be another method that returns that rather than faking out and corrupting the functionality of this method.
(0026515)
vegaslon   
2014-07-19 09:39   
I have submitted another patch that does what I believe you asked to be done.
(0029242)
aiaustin   
2015-08-18 09:34   
Resolved issue marked as closed. Please open a new issue and refer to this one if problems arise.