opensim-moco icon indicating copy to clipboard operation
opensim-moco copied to clipboard

Add example/documentation for using IK solution with MocoStateTrackingGoal

Open chrisdembia opened this issue 4 years ago • 8 comments

Main issues are changing column labels and converting to radians.

chrisdembia avatar Mar 26 '20 19:03 chrisdembia

This was a repeated issue during the March 2020 workshop.

chrisdembia avatar Mar 26 '20 19:03 chrisdembia

Having some feedback/output on what's being done behind the scenes could also help.

For instance, the radians issue was found because of the tracke_states.sto file. While the input had inDegrees=yes, the output file had inDegrees=no even though all the values were the same.

carmichaelong avatar Mar 29 '20 17:03 carmichaelong

I'm not sure what is meant by the output file. Was that the MocoSolution STO file?

chrisdembia avatar Mar 29 '20 17:03 chrisdembia

Oops, output file = tracked_states.sto file. That's a typo.

carmichaelong avatar Mar 29 '20 17:03 carmichaelong

Oh! That's generated by MocoTrack during preprocessing. I forgot. That's a useful clue. This is a high-priority issue.

I do not understand where the issue is, given that here is where process() is invoked:

https://github.com/opensim-org/opensim-moco/blob/c8335c878548b137f9eeb159bdc3c647d1a96060/Moco/Moco/MocoTrack.cpp#L150-L151

and this is where the angle conversion happens in process():

https://github.com/opensim-org/opensim-moco/blob/c8335c878548b137f9eeb159bdc3c647d1a96060/Moco/Moco/Common/TableProcessor.h#L103-L106

chrisdembia avatar Mar 29 '20 17:03 chrisdembia

@psbiomech have you used IK solutions as the state tracking reference in Moco?

chrisdembia avatar Mar 29 '20 18:03 chrisdembia

@carmichaelong it might be helpful to have a set of files that reproduce the problem. It probably warrants it's own GitHub issue; if you have some time to reproduce and create the issue that would be helpful.

nickbianco avatar Mar 30 '20 00:03 nickbianco

Definitely. Will plan on trying to get a minimal example for C++ this week.

carmichaelong avatar Mar 30 '20 05:03 carmichaelong