search

DAIRLab / dairlib

MIT License
65 stars 26 forks source link

Fix a bug in osc tracking osc logging #330

Closed yminchen closed 1 year ago

yminchen commented 1 year ago

The osc tracking cost order is inconsistent with the name's, because we save the names using vector.push_back while saving the cost value using indices

This change is Reviewable

yminchen commented 1 year ago

push_back + indexing caused issue when there are inactive tracking data. That is, push_back skips inactive tracking data but indexing does not.

Yes #325 fixes this issue. I'm closing this PR