nrtb-core team mailing list archive

[Rick Stovall <fpstovall@xxxxxxxxx>]

Public bug reported:

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.

** Affects: nrtb
     Importance: Medium
     Assignee: Rick Stovall (fpstovall)
         Status: Triaged

** Changed in: nrtb
    Milestone: None => sprint-005a

-- 
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:
  Triaged

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