Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
dgupta7 opened this issue Feb 17, 2023 · 3 comments
Open

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

dgupta7 opened this issue Feb 17, 2023 · 3 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@dgupta7
Copy link
Collaborator

dgupta7 commented Feb 17, 2023

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

  • Windows 10
  • MATLAB R2021a
  • OpenSim 4.3
  • CasADi 3.5.5
  • Microsoft Visual Studio: Visual Studio 15 2017 Win64
  • CMake 3.24.1
@dgupta7 dgupta7 added bug Something isn't working conditional bug and removed bug Something isn't working labels Mar 23, 2023
@VDB-Bram
Copy link
Collaborator

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

@dgupta7
Copy link
Collaborator Author

dgupta7 commented Mar 29, 2023

It might be useful to do both

@VDB-Bram
Copy link
Collaborator

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.

@Lars-DHondt-KUL Lars-DHondt-KUL added bug Something isn't working and removed conditional bug labels Sep 11, 2023
@Lars-DHondt-KUL Lars-DHondt-KUL added this to the v1.2.0 milestone Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants