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

[v14 ready] Latexify improvement #742

Merged
merged 4 commits into from
Feb 2, 2024
Merged

Conversation

TorkelE
Copy link
Member

@TorkelE TorkelE commented Dec 7, 2023

Makes some improvements to make Latexify automatically use our new expand_registered_functions function to expand mm and similar. Now this is done by default for reaction, ode, and sde forms. Also, setting the kwarg expand_functions=false disables this.

using Catalyst, Latexify
rn = @reaction_network begin
    (mm(x,v,K),d), 0 <--> X
end
latexify(rn, form=:ode, expand_functions=false)

@TorkelE TorkelE changed the base branch from master to Catalyst_version_14 February 2, 2024 14:53
@TorkelE TorkelE changed the title Latexify improvement [v14 ready] Latexify improvement Feb 2, 2024
Copy link
Member

@isaacsas isaacsas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment, otherwise LGTM.

docs/src/api/catalyst_api.md Outdated Show resolved Hide resolved
Co-authored-by: Sam Isaacson <isaacsas@users.noreply.github.com>
@TorkelE TorkelE merged commit 315bc3c into Catalyst_version_14 Feb 2, 2024
6 of 7 checks passed
@TorkelE TorkelE deleted the Latexify_updates branch February 2, 2024 16:23
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