Skip to content

Fix PySCF startup timeout#592

Open
fallintoplace wants to merge 1 commit into
NVIDIA:mainfrom
fallintoplace:fix-pyscf-startup-timeout
Open

Fix PySCF startup timeout#592
fallintoplace wants to merge 1 commit into
NVIDIA:mainfrom
fallintoplace:fix-pyscf-startup-timeout

Conversation

@fallintoplace

Copy link
Copy Markdown

Summary

  • Require PySCF health checks to return status: available before the driver reports ready.
  • Keep startup polling through REST connection failures until the timeout expires.
  • Drop unused local variables in the startup path.

Why

When RestClient::get() throws while the PySCF server is still starting or unreachable, the previous loop continued before incrementing or checking the timeout counter. That could hang forever on connection failures. The availability checks also treated malformed or non-ready responses as available.

Validation

  • git diff --check

Not run locally: full solvers build, because this checkout does not have a configured CUDA-Q/LLVM build environment.

Signed-off-by: Minh Vu <vuhoangminh97@gmail.com>
@copy-pr-bot

copy-pr-bot Bot commented Jun 7, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

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.

1 participant