← Back to team overview

nrtb-core team mailing list archive

[Bug 1272383] Re: Clean up force application code.

 

** Changed in: nrtb/alpha
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of NRTB
Core, which is subscribed to New Real Time Battle.
https://bugs.launchpad.net/bugs/1272383

Title:
  Clean up force application code.

Status in The New Real Time Battle Project:
  Fix Released
Status in New Real Time Battle alpha series:
  Fix Released

Bug description:
  In cpp/sim_engine/base_object/base_object.cpp:

  bool base_object::apply(int time, float quanta)
  {
   (....)
    
    // existing code (too ugly to live)
    velocity += (a * quanta) + (accel_mod * quanta);  
    rotation += (ra * quanta) + (torque_mod * quanta);
    
    // should be
    velocity += (a + accel_mod) * quanta;  
    rotation += (ra + torque_mod) * quanta;

  (....)
  };

  This new code will be easier to understand and may run better.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nrtb/+bug/1272383/+subscriptions


References