-
Notifications
You must be signed in to change notification settings - Fork 186
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
Error running spykingcircus, due to space in recording file path #3413
Comments
Generally speaking, spaces in paths are a nightmare to deal with.. I would try to enforce it ad much as I could, but feel free to also make a PR with added quotes. I think that being extra cautious avuto this doesn't hurt! |
In trying to fix this, I'm working from a local version to run spykingcircus in docker, using
|
Running spykingcircus in docker mode (probably in regular mode as well) results in an error if there is a space in the path.
This probably relates to the shell command not putting the path in quotes:
spikeinterface/src/spikeinterface/sorters/external/spyking_circus.py
Lines 157 to 169 in 082bb39
Is it mandatory that the path has no spaces? I can not really enforce this from an automation perspective in my own experiment pipeline...
If it's acceptable to put path in quotes (and this wouldn't break anything else) then I'm willing to make a PR for this.
The text was updated successfully, but these errors were encountered: