Skip to content

ci(documentation): add runner input for GPU-queue doc builds#70

Merged
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:docs-gpu-runner
Jun 7, 2026
Merged

ci(documentation): add runner input for GPU-queue doc builds#70
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:docs-gpu-runner

Conversation

@ChrisRackauckas-Claude
Copy link
Copy Markdown

Adds a runner input to documentation.yml (mirrors tests.yml): a JSON-encoded runs-on value (e.g. ["self-hosted","Linux","X64","gpu"]) that overrides self-hosted/os when non-empty. This lets repos whose docs require a GPU build via the central documentation.yml@v1 caller on a GPU queue, instead of a bespoke GPU.yml/ci.yml.

runs-on becomes ${{ inputs.runner != '' && fromJson(inputs.runner) || (inputs.self-hosted && 'self-hosted' || inputs.os) }}. Backward-compatible (default "" preserves current behavior).

After merge + v1 retag, the GPU-docs repos (HighDimPDE, ModelingToolkit, Optimization, ReservoirComputing, SciMLSensitivity) can switch their bespoke doc builds to this caller with runner: ....

Ignore until reviewed by @ChrisRackauckas.

🤖 Generated with Claude Code

Mirrors tests.yml: a JSON-encoded `runner` input (array of labels, e.g.
["self-hosted","Linux","X64","gpu"]) that overrides self-hosted/os when
non-empty, so docs that require a GPU can build via the central
documentation.yml@v1 caller (runs-on uses fromJson(runner)) instead of a
bespoke GPU.yml. Backward-compatible: default "" preserves
self-hosted/os behavior for all existing callers.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas marked this pull request as ready for review June 7, 2026 12:09
@ChrisRackauckas ChrisRackauckas merged commit dc62281 into SciML:master Jun 7, 2026
2 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