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 uci.Engine.Getpid() #124

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add uci.Engine.Getpid() #124

wants to merge 1 commit into from

Conversation

mikeb26
Copy link
Contributor

@mikeb26 mikeb26 commented May 26, 2023

This change adds a Renice() method receiver to uci.Engine. This enables callers to specify that the engine should be run at a lower priority. In practice this is helpful in improving system responsiveness when running the engine with
high thread counts and/or larger hash sizes.

(cherry picked from commit 92c1903)

@notnil
Copy link
Owner

notnil commented Nov 25, 2024

Maybe just surfacing the PID is a bit cleaner and allows for a multitude of performance monitoring use cases

This change adds a Getpid() method receiver to uci.Engine. This
enables callers to interact with the engine process directly. For
example callers may renice the process to allow the engine to run
at a lower priority. In practice this is helpful in improving system
responsiveness when running the engine with high thread counts and/or
larger hash sizes.
@mikeb26 mikeb26 changed the title Add uci.Engine.Renice() Add uci.Engine.Getpid() Nov 26, 2024
@mikeb26
Copy link
Contributor Author

mikeb26 commented Nov 26, 2024

Maybe just surfacing the PID is a bit cleaner and allows for a multitude of performance monitoring use cases

done.

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