MSO API doesn't handle spaces in directories

Logic MSO
1

When the save_dir contains spaces in the path, the current MSO.capture() function does not wrap the CLI argument in quotes so it fails.

Need to update the MSO.capture() method to be

...            
            self.wrapper.run(
                "cli",
                "capture",
                "record-data",
                "--json",
                "--args-path",
                f"'{str(record_options_path.absolute())}'",
                "--out",
                f"'{str(result_path.absolute())}'",
                timeout_secs=timeout_secs,
            )
...

Side note is the documentation ( Contributing — Saleae MSO API Documentation ) says that the API repo is on github but it appears to be private so PRs and issues can’t be created there.

2

Sorry for the confusion with the private GitHub repo link! The API documentation link you shared is from an older version (v0.5.0). Can I know how you reached that? We should probably take that documentation page down.

Here is the latest MSO API documentation. The MSO API repo is meant to be private.

We’ll take a look at the save_dir bug as well. Thanks for reporting that.