Skip to content

[Issue#213] Add tests to statesp.zero #260

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.

Sign up for GitHub

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

Jump to bottom
Merged
murrayrm merged 4 commits into from
Jan 4, 2019
Merged

[Issue#213] Add tests to statesp.zero #260

murrayrm merged 4 commits into from
Jan 4, 2019

Conversation

@don4get
Copy link
Contributor

@don4get don4get commented Dec 30, 2018

Hello again,

here are more tests for statesp.zero as asked in #213.

I also made small refactor including pep8 recommendations.

Have a good night!

@coveralls
Copy link

coveralls commented Dec 30, 2018 •
edited
Loading

Coverage Status

Coverage remained the same at 77.953% when pulling 6873c8a on don4get:feature/add-zero-tests into 0d94329 on python-control:master.

@don4get don4get force-pushed the feature/add-zero-tests branch from d9f5732 to 6873c8a Compare December 30, 2018 21:42
@don4get don4get changed the title Feature/add zero tests https://github.com/python-control/python-control/issues/213 Add tests to zero Dec 31, 2018
@don4get don4get changed the title https://github.com/python-control/python-control/issues/213 Add tests to zero https://github.com/python-control/python-control/issues/213 Dec 31, 2018
@don4get don4get changed the title https://github.com/python-control/python-control/issues/213 Add tests to statesp.zero Dec 31, 2018
@don4get don4get changed the title Add tests to statesp.zero [Issue#213] Add tests to statesp.zero Dec 31, 2018
@murrayrm murrayrm merged commit eeb7c2f into python-control:master Jan 4, 2019
@murrayrm murrayrm mentioned this pull request Jan 4, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@don4get @coveralls @murrayrm