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 UpdateFunctionsOfTime action #6264

Merged
merged 3 commits into from
Nov 17, 2024

Conversation

nikwit
Copy link
Contributor

@nikwit nikwit commented Sep 3, 2024

Adds the action that adjusts the functions of time according to the motion of the particle.

depends on #6255 #6261

@nikwit nikwit added the dependent Needs a different PR to be merged in first label Sep 3, 2024
@nikwit nikwit removed the dependent Needs a different PR to be merged in first label Nov 6, 2024
@nikwit nikwit requested a review from knelli2 November 6, 2024 16:16
Copy link
Contributor

@knelli2 knelli2 left a comment

Choose a reason for hiding this comment

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

Except for the clang-tidy errors and some missing tags in the tests, looks good.

@nikwit
Copy link
Contributor Author

nikwit commented Nov 13, 2024

My bad, I accidentally left out some of the changes. I pushed a new fixup.

Copy link
Contributor

@knelli2 knelli2 left a comment

Choose a reason for hiding this comment

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

Looks good, squash

@nikwit
Copy link
Contributor Author

nikwit commented Nov 13, 2024

done, thanks for the review!

@nikwit nikwit requested a review from knelli2 November 16, 2024 12:17
@knelli2 knelli2 merged commit c05c009 into sxs-collaboration:develop Nov 17, 2024
23 checks passed
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