Skip to content

Fix cmdstan path for self-hosted runners#366

Merged
ChrisRackauckas merged 1 commit intoSciML:masterfrom
ChrisRackauckas-Claude:fix-stan-cmdstan-path
Jan 6, 2026
Merged

Fix cmdstan path for self-hosted runners#366
ChrisRackauckas merged 1 commit intoSciML:masterfrom
ChrisRackauckas-Claude:fix-stan-cmdstan-path

Conversation

@ChrisRackauckas-Claude
Copy link

Summary

  • Use $HOME instead of hardcoded /home/runner/ path for JULIA_CMDSTAN_HOME to support both GitHub-hosted and self-hosted runners
  • The Stan tests were failing on self-hosted runners because the hardcoded path /home/runner/cmdstan-2.34.1/ doesn't exist there

Changes

  • stan.sh: Use $HOME/cmdstan-2.34.1/ and export to $GITHUB_ENV for subsequent steps
  • CI.yml: Remove hardcoded path, rely on stan.sh export via $GITHUB_ENV
  • Documentation.yml: Remove hardcoded path, rely on stan.sh export via $GITHUB_ENV

Test plan

  • Verify Stan tests pass on GitHub-hosted runners (ubuntu-latest where $HOME=/home/runner/)
  • Verify Stan tests pass on self-hosted runners (where $HOME varies)

🤖 Generated with Claude Code

Use $HOME instead of hardcoded /home/runner/ path for JULIA_CMDSTAN_HOME
to support both GitHub-hosted and self-hosted runners.

Changes:
- stan.sh: Use $HOME/cmdstan-2.34.1/ and export to GITHUB_ENV
- CI.yml: Remove hardcoded path, rely on stan.sh export
- Documentation.yml: Remove hardcoded path, rely on stan.sh export

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas merged commit bf9433f into SciML:master Jan 6, 2026
8 of 10 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.

3 participants