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

Optionally disable pipeline halt on SIGINT #154

Merged
merged 1 commit into from
Apr 2, 2024
Merged

Optionally disable pipeline halt on SIGINT #154

merged 1 commit into from
Apr 2, 2024

Conversation

arahlin
Copy link
Member

@arahlin arahlin commented Apr 1, 2024

Optional argument signal_halt for the G3Pipeline.Run() method. By default, this is set to True to enable current behavior, namely that a pipeline is halted (via the halt_processing flag) and terminates after the the current frame reaches the end of the pipeline. If False, this signal handling is disabled, and it is then up to the user to ensure that a running pipeline is properly halted on interrupt.

Optional argument `signal_halt` for the `G3Pipeline.Run()` method.  By default,
this is set to True to enable current behavior, namely that a pipeline is halted
(via the `halt_processing` flag) and terminates after the the current frame
reaches the end of the pipeline.  If False, this signal handling is disabled,
and it is then up to the user to ensure that a running pipeline is properly
halted on interrupt.
@arahlin arahlin requested a review from nwhitehorn April 1, 2024 23:57
@arahlin arahlin self-assigned this Apr 1, 2024
Copy link
Member

@nwhitehorn nwhitehorn left a comment

Choose a reason for hiding this comment

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

Thanks!

@arahlin arahlin merged commit 96a6364 into master Apr 2, 2024
1 check passed
@arahlin arahlin deleted the signal_halt branch April 2, 2024 01:32
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