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

Implement SMoment #817

Closed
wants to merge 2 commits into from
Closed

Implement SMoment #817

wants to merge 2 commits into from

Conversation

stelmo
Copy link
Collaborator

@stelmo stelmo commented Jan 29, 2024

Implement SMoment to reduce variable creation associated with normal enzyme models.

Some small fixes to enzyme models as well.

src/builders/enzymes.jl Outdated Show resolved Hide resolved
src/COBREXA.jl Outdated Show resolved Hide resolved
src/builders/enzymes.jl Show resolved Hide resolved
src/builders/interface.jl Outdated Show resolved Hide resolved
src/frontend/enzymes.jl Outdated Show resolved Hide resolved
@stelmo stelmo requested a review from exaexa January 29, 2024 10:35
@stelmo
Copy link
Collaborator Author

stelmo commented Jan 29, 2024

/format

Copy link
Contributor

✔️ Auto-formatting triggered by this comment succeeded, commited as 6fed64a

github-actions bot pushed a commit that referenced this pull request Jan 29, 2024
triggered by @stelmo on PR #817
Copy link
Collaborator

@exaexa exaexa left a comment

Choose a reason for hiding this comment

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

I can finish this tomorrow when merging

src/builders/enzymes.jl Show resolved Hide resolved
src/builders/enzymes.jl Show resolved Hide resolved
src/builders/enzymes.jl Show resolved Hide resolved
src/builders/enzymes.jl Show resolved Hide resolved
Base automatically changed from mk-january-cleanup to next January 29, 2024 21:01
@stelmo
Copy link
Collaborator Author

stelmo commented Feb 2, 2024

in v2

@stelmo stelmo closed this Feb 2, 2024
@stelmo stelmo deleted the sew-smoment branch February 2, 2024 19:31
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