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

Add enzyme constrained models #805

Merged
merged 48 commits into from
Dec 28, 2023
Merged

Add enzyme constrained models #805

merged 48 commits into from
Dec 28, 2023

Conversation

stelmo
Copy link
Collaborator

@stelmo stelmo commented Dec 9, 2023

Add enzyme constrained models

@exaexa exaexa changed the base branch from sew-ct-update to next December 10, 2023 16:47
@stelmo
Copy link
Collaborator Author

stelmo commented Dec 14, 2023

/format

triggered by @stelmo on PR #805
Copy link
Contributor

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

@stelmo stelmo requested a review from exaexa December 14, 2023 16:02
@stelmo
Copy link
Collaborator Author

stelmo commented Dec 17, 2023

/format

@stelmo
Copy link
Collaborator Author

stelmo commented Dec 27, 2023

Hmm the tests pass locally for me. I think its a CI issue with the branch of CTs with the fixed variables function. Anyways, gonna address your comments now

@stelmo
Copy link
Collaborator Author

stelmo commented Dec 27, 2023

Reviews are partially implemented, anything unresolved I left for you @exaexa

@stelmo
Copy link
Collaborator Author

stelmo commented Dec 27, 2023

/format

triggered by @stelmo on PR #805
Copy link
Contributor

✔️ Auto-formatting triggered by this comment succeeded, commited as 11b52ee

@stelmo stelmo marked this pull request as ready for review December 28, 2023 13:09
@stelmo stelmo requested a review from exaexa December 28, 2023 13:10
@stelmo stelmo merged commit 2a26bf9 into next Dec 28, 2023
1 check passed
@stelmo stelmo deleted the sew-enzyme-models branch December 28, 2023 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature/functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants