Skip to content

Conversation

Drewniok
Copy link
Collaborator

@Drewniok Drewniok commented Feb 1, 2025

Description

This PR adds the option to set a timeout in QuickSim to avoid endless simulations.

Checklist:

  • The pull request only contains commits that are related to it.
  • I have added appropriate tests and documentation.
  • I have added a changelog entry.
  • I have created/adjusted the Python bindings for any new or updated functionality.
  • I have made sure that all CI jobs on GitHub pass.
  • The pull request introduces no new warnings and follows the project's style guidelines.

@Drewniok Drewniok added the enhancement New feature or request label Feb 1, 2025
@Drewniok Drewniok self-assigned this Feb 1, 2025
@Drewniok Drewniok changed the title ✨ Added timout for _QuickSim_ ✨ Added timout for QuickSim Feb 1, 2025
@Drewniok Drewniok changed the title ✨ Added timout for QuickSim ✨ Added timout for *QuickSim* Feb 1, 2025
@Drewniok Drewniok changed the title ✨ Added timout for *QuickSim* ✨ Added timout for `QuickSim Feb 1, 2025
@Drewniok Drewniok changed the title ✨ Added timout for `QuickSim ✨ Added timout for QuickSim Feb 1, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

There were too many comments to post at once. Showing the first 25 out of 145. Check the log or trigger a new build to see more.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

There were too many comments to post at once. Showing the first 25 out of 120. Check the log or trigger a new build to see more.

@Drewniok Drewniok requested a review from marcelwa February 28, 2025 17:06
libs/json Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Manual pull requests should not update the submodules

libs/pybind11 Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

Manual pull requests should not update the submodules

@Drewniok Drewniok requested a review from marcelwa March 2, 2025 14:39
@marcelwa marcelwa merged commit 340e31e into cda-tum:main Mar 3, 2025
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants