Skip to content

remove partial rendering#86

Merged
chomosuke merged 1 commit intomasterfrom
remove-partial-rendering
Apr 12, 2025
Merged

remove partial rendering#86
chomosuke merged 1 commit intomasterfrom
remove-partial-rendering

Conversation

@chomosuke
Copy link
Copy Markdown
Owner

I forgot why I added this, it's clearly experimental and it's causing some people problem.

I believe the amount time people spent waiting for extra time of rendering with this option off will be less than the amount of time people spent trying to figure out what's wrong with this plugin.

@atlanswer Wondering what is your thought on this PR.

I forgot why I added this, it's clearly experimental and it's causing
some people problem.

I believe the amount time people spent waiting for extra time of
rendering with this option off will be less than the amount of time
people spent trying to figure out what's wrong with this plugin.
@atlanswer
Copy link
Copy Markdown
Contributor

atlanswer commented Apr 6, 2025

Don't enable it by default should be enough IMO. Typst is already way faster than the standard LuaLaTeX and that's what matters. Power users will try to find ways to make Tinymist preview faster if they need it and compare the speed and result. There's no benchmark on partial rendering, and several issues are opened in the Tinymist repo that have it enabled suggest not making it the default option.

@chomosuke
Copy link
Copy Markdown
Owner Author

Thank you for your opinion, and I concur

@chomosuke chomosuke merged commit 2503b18 into master Apr 12, 2025
@chomosuke chomosuke deleted the remove-partial-rendering branch April 20, 2025 04:16
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