search

KULeuvenNeuromechanics / PredSim

Generate predictive simulations of human locomotion
MIT License
26 stars 48 forks source link

Long file/path name of osim files could give errors #66

Open dgupta7 opened 1 year ago

dgupta7 commented 1 year ago

Before submitting a bug report, read the error messages in MATLAB. Some errors are there to inform you about inconsistent inputs for certain settings.

Describe the bug I ran the Main.m. I got an error somewhere in the osim2dll step. When I ran the exact same code with a shorter filename, it works.

osim path for which it works: C:\GBW_MyPrograms\PredSim_Jan\PredSim\Subjects\CP3_noShankPersonalization_cpp1\CP3_noShankPersonalization_cpp1.osim

osim path for which it does not work: C:\GBW_MyPrograms\PredSim_Jan\PredSim\Subjects\CP3_noShankPersonalization_conditionalPathPoint1\CP3_noShankPersonalization_conditionalPathPoint1.osim

They are the same .osim files, just with different names.

In the log file (attached), it says twice that "The fully qualified file name must be less than 260 characters"

To Reproduce Steps to reproduce the behavior:

  1. Which script did you run?: Main.m
  2. What settings did you use?: specific to the study, but I don't think it matters here since I am getting error in the dll2osim step
  3. What kind of OpenSim model did you use? Link to generic model, attach specific model, or describe model (bodies, joints, coordinates) if it is confidential.: mri based model of a child with CP

Log file CP3_noShankPersonalization_conditionalPathPoint1_v1_log.txt

Software versions

VDB-Bram commented 1 year ago

should we put in a warning in the code or a description in the readme?

dgupta7 commented 1 year ago

It might be useful to do both

VDB-Bram commented 1 year ago

Longest path has this structure? path_to_opensimAD_buildExpressionGraph/F_modelName/F_modelName.dir/RelWithDebInfo/F_modelName.vcxproj.CopyComplete

To add in PredSim code: Check number of characters given the above structure, give error before osim to dll conversion if this exceeds 260.

Since the number of charachters will be dependent on where PredSim is installed, I think it is difficult to write something in the Readme.