Skip to content

Bugfix: sh=2.0.0 changed the return type from sh.RunningCommand to str#32

Open
boeddeker wants to merge 1 commit intofgnt:masterfrom
boeddeker:master
Open

Bugfix: sh=2.0.0 changed the return type from sh.RunningCommand to str#32
boeddeker wants to merge 1 commit intofgnt:masterfrom
boeddeker:master

Conversation

@boeddeker
Copy link
Member

Fix for #31

Before sh=2.0.0, the return type of a command was sh.RunningCommand and it was interpreted as input to another command.

Changed the code to use subprocess.

@boeddeker boeddeker requested review from TCord and thequilo April 29, 2024 08:42
@boeddeker
Copy link
Member Author

There are some more issues with recent packages (numpy and scipy), that have to be solved first.

import os
import re
import tempfile
import subprocess
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

import sh is now unused

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also fix the imports for the scipy window functions (in the stft)? Currently, some Import tests still fail.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants