Skip to content
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

discrete time LaTeX repr of StateSpace systems #609

Merged
merged 3 commits into from Apr 12, 2021

Conversation

bnavigator
Copy link
Contributor

ss-dt-latex-partitioned

ss-dt-latex-separate

@bnavigator
Copy link
Contributor Author

@roryyorke

@coveralls
Copy link

coveralls commented Apr 11, 2021

Coverage Status

Coverage increased (+0.02%) to 89.471% when pulling 4608593 on bnavigator:latex-dt into 15dc5ce on python-control:master.

@sawyerbfuller
Copy link
Contributor

Sweet! I’ve been thinking this needed attention.

Related comment: I think it might be hard to tell if dt=unspecified means dt=none (no timebase) or dt=True (unspecified timebase).

What do you think about calling it dt=True instead? Benefit: can unambiguously tell it is a discrete time system by reading it in your head.

@bnavigator
Copy link
Contributor Author

What do you think about calling it dt=True instead? Benefit: can unambiguously tell it is a discrete time system by reading it in your head.

I like that!

@murrayrm
Copy link
Member

LGTM

@bnavigator bnavigator merged commit d1f3bca into python-control:master Apr 12, 2021
@murrayrm murrayrm added this to the 0.9.1 milestone Dec 30, 2021
@bnavigator bnavigator deleted the latex-dt branch February 18, 2024 20:30
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.

None yet

4 participants