artofnothingness / mppic

MIT License
84 stars 19 forks source link

improves performance #73

Closed SteveMacenski closed 2 years ago

SteveMacenski commented 2 years ago

Total, these 2 changes get us back about 14% -- which is kind of nuts

This leaves integrateStateVelocities's concatenation at a total of 21% of the total run time (and integrateStateVelocities in total is 26%).

I can't tell exactly where the 46-26=20% is left under the trajectory generation process, but it points to generateNoisedControls as a function, so I'm making an educated guess that the concatenation there is a big contributor to it since those numbers are consistent with the overhead in integrateStateVelocities and those are the only 2 places that concats are used anywhere in the program.