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

Cut up long vectors for inline display #1

Merged
merged 1 commit into from
Sep 3, 2015

Conversation

nilshg
Copy link
Owner

@nilshg nilshg commented Sep 1, 2015

Prevents Atom from freezing if one accidentally evaluates a super long vector.

[Sorry, this is the best refactoring I could come up with, still three calls to render but at least I got rid of one of the two Tree constructors]

Prevents Atom from freezing if one accidentally evaluates a super long vector.
@MikeInnes
Copy link

That's fine, exactly what I was thinking of.

Unfortunately I can't merge this PR since it's not my repo ;)

nilshg added a commit that referenced this pull request Sep 3, 2015
Cut up long vectors for inline display
@nilshg nilshg merged commit 9a3cb19 into master Sep 3, 2015
@nilshg nilshg deleted the nilshg-display-long-vectors branch September 3, 2015 09:01
@nilshg
Copy link
Owner Author

nilshg commented Sep 3, 2015

Sorry, my mistake! Gave it another go, hope this works!

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.

2 participants