Skip to content

Conversation

@bnavigator
Copy link
Contributor

Another take on #533

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.711% when pulling 25114f5 on bnavigator:pytest-timeout into d75c395 on python-control:master.

@murrayrm
Copy link
Member

murrayrm commented Feb 7, 2021

This looks more than #535 to me. I suggest we merge this and drop #535. Will wait for 8 hours or so in case anyone wants to chime in.

@bnavigator
Copy link
Contributor Author

Okay. I just want to make sure you are okay with the fact that we add another test dependency.

@bnavigator bnavigator linked an issue Feb 7, 2021 that may be closed by this pull request
@murrayrm murrayrm merged commit 0320595 into python-control:master Feb 7, 2021
@murrayrm murrayrm added this to the 0.9.0 milestone Mar 20, 2021
@bnavigator bnavigator deleted the pytest-timeout branch February 18, 2024 20:29
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.

Root locus timeout on Windows: no SIGALRM

3 participants