Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add _repr_latex_ for Jupyter notebooks#276

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
murrayrm merged 3 commits intopython-control:masterfromalchemyst:master
Apr 7, 2019

Conversation

alchemyst
Copy link
Contributor

@alchemystalchemyst commentedFeb 24, 2019
edited
Loading

This allows better representation inside the browser:

image

ilayn and don4get reacted with thumbs up emoji
@coveralls
Copy link

coveralls commentedFeb 24, 2019
edited
Loading

Coverage Status

Coverage decreased (-0.5%) to 77.92% when pulling6cbe066 on alchemyst:master intoe4e7a0d on python-control:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-13.5%) to 64.872% when pulling426215e on alchemyst:master into31e6944 on python-control:master.

@murrayrm
Copy link
Member

Change looks fine, but getting a Travis CI error.

It would be nice to put in this type of functionality for the other LTI subtypes (StateSpace, FRD, etc). Any chance you have the time and inclination to do that as well?

@alchemyst
Copy link
ContributorAuthor

I'd be happy to take a run at it, but perhaps we could start by opening issues for the others? I'd specifically like to know what format you'd prefer for the state space.

@murrayrm
Copy link
Member

@alchemyst Can you merge changes from latest master to get Travis CI working again (see PR#280 for more info). Thanks.

@alchemyst
Copy link
ContributorAuthor

@murrayrm

@repagh
Copy link
Member

Nice, looks good.
Would it be worthwhile to build the outstr as a list, and at the end return the string with
''.join(outstr)
I expect that to be faster.

@alchemyst
Copy link
ContributorAuthor

@repagh Done.

My initial implementation was a copy-paste of the current__str__. Perhaps we should add an issue for someone to make the same changes there?

@repagh
Copy link
Member

@murrayrm I think this looks OK to merge

@murrayrmmurrayrm added this to the0.8.2 milestoneApr 6, 2019
@murrayrmmurrayrm merged commitc30746a intopython-control:masterApr 7, 2019
Sign up for freeto 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
0.8.2
Development

Successfully merging this pull request may close these issues.

4 participants
@alchemyst@coveralls@murrayrm@repagh

[8]ページ先頭

©2009-2025 Movatter.jp