fourMs / MGT-python

Musical Gestures Toolbox for Python
https://www.uio.no/ritmo/english/research/labs/fourms/downloads/software/musicalgesturestoolbox/mgt-python/index.html
GNU General Public License v3.0
52 stars 11 forks source link

Problems downloading models for pose #299

Closed alexarje closed 11 months ago

alexarje commented 1 year ago

When trying to use the pose method, I found that the download chokes after inserting the password:

ValueError: could not convert string to float: 's/' Screenshot from 2023-08-03 10-17-36

I think this may be because I have spaces in my password? May be worth taking that into account in the code.

joachimpoutaraud commented 12 months ago

Thanks for reporting this issue! I think the problem was just that your password was added to the input used for defining the answer variable (i.e. 'y' or 'n'). I have updated the Python input() function with input()[0] so that it will only take the first letter of the string variable in case the user needs to enter a password before downloading the model.

alexarje commented 11 months ago

I still get an error when trying this one. Do you have any ideas on what else could be wrong?

Screenshot from 2023-10-04 14-22-57

joachimpoutaraud commented 11 months ago

Yes I think I found the problem this time (a tricky one). It was related to the shell script (getCOCO_remote.sh) which caused an error because of the presence of an OD character in the url. As a result, it wasn't possible to connect to the url where the model is stored in order to be downloaded. The OD character is a carriage return, part of the CRLF sequence that Windows uses for line endings just to be different as usual.

I have updated the shell file getCOCO_remote.sh and tested to download the model using Windows Subsystem for Linux (WSL) and it worked fine.