Skip to content

[CI] Add GHA job to test downstream GB-25 #1197

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.

Sign up for GitHub

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

Merged
wsmoses merged 32 commits into from
Jul 31, 2025
Merged

[CI] Add GHA job to test downstream GB-25 #1197

Changes from 1 commit
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
4048dc9
[CI] Add GHA job to test downstream GB-25
giordano Jul 29, 2025
2907d3e
[CI] Temporarily disable repository cache
giordano Jul 29, 2025
6cd7378
Revert "[CI] Temporarily disable repository cache"
giordano Jul 29, 2025
7c8ab16
[CI] Upload build directory for debugging
giordano Jul 29, 2025
11ea4c1
Update test-gb-25.yml
wsmoses Jul 29, 2025
076d3b8
Update test-gb-25.yml
wsmoses Jul 29, 2025
0ec4c8e
Use tmate for debugging
giordano Jul 29, 2025
e6fcb7f
Update test-gb-25.yml
wsmoses Jul 29, 2025
10290d8
Update test-gb-25.yml
wsmoses Jul 29, 2025
3a58f96
Update test-gb-25.yml
wsmoses Jul 29, 2025
16bb5c5
Update test-gb-25.yml
wsmoses Jul 29, 2025
622cb49
[CI] Change Enzyme-JAX commit used for compiling libReactantExtra
giordano Jul 29, 2025
853e142
Update test-gb-25.yml
wsmoses Jul 29, 2025
e652e15
[CI] Set clang as compiler
giordano Jul 29, 2025
9f4cbb2
[CI] Restore bazel cache
giordano Jul 29, 2025
89bf7af
[CI] Run tmate only on failure (will eventually remove it)
giordano Jul 29, 2025
8270a24
[CI] Checkout GB-25 repo
giordano Jul 30, 2025
af0beaa
Update test-gb-25.yml
giordano Jul 30, 2025
ff5025a
[CI] Explicitly copy preferences file
giordano Jul 30, 2025
e87bb97
Update test-gb-25.yml
giordano Jul 30, 2025
5c091dc
[CI] Run GB-25 simulation
giordano Jul 30, 2025
bbacbd6
[CI] Add timeouts
giordano Jul 30, 2025
c57c507
[CI] Disable earlyoom
giordano Jul 30, 2025
e3d81e5
[CI] Upload artifacts
giordano Jul 30, 2025
7d8adee
[CI] Also dump all XLA passes
giordano Jul 30, 2025
1be36f4
Longer timeouts
giordano Jul 30, 2025
6965da3
[CI] Add code (commented out) to change XLA commit
giordano Jul 30, 2025
cf9f1a2
[CI] Also remove loading of XLA from JAX
giordano Jul 30, 2025
09919a3
[CI] Add XLA, GB-25, and Reactant commits as matrix elements
giordano Jul 30, 2025
6d10a77
Update test-gb-25.yml
wsmoses Jul 30, 2025
4cfd560
[CI] Fix paths of artifacts to upload
giordano Jul 30, 2025
f1afb2e
Update test-gb-25.yml
wsmoses Jul 31, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .github/workflows/test-gb-25.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,9 @@ jobs:
# Modify commit Enzyme-JAX commit
sed -i.bak 's/ENZYMEXLA_COMMIT = ".*"/ENZYMEXLA_COMMIT = "${{ github.sha }}"/' ReactantExtra/WORKSPACE

# Modify XLA commit
# sed -E -i.bak -e 's/(# )?XLA_COMMIT = ".*"/XLA_COMMIT = "0123456789abcdef0123456789abcdef01234567"/' -e 's/(# )?XLA_SHA256 = ""/XLA_SHA256 = ""/' ReactantExtra/WORKSPACE

@wsmoses wsmoses Jul 30, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Two things:

  1. we also need to comment out or delete the load of the xla commit from Jax
  2. we should match and hash

@wsmoses wsmoses Jul 30, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah jk I see you do 2

@wsmoses wsmoses Jul 30, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should also add a series of targets, like
Xla_hash:

  • ""
  • "abcd..."

To the github actions yml, and the have it use the hash if non empty

@giordano giordano Jul 30, 2025

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Uhm, not sure how you mean exactly. Do you have an example?

@giordano giordano Jul 30, 2025

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ok, done in 09919a3 (#1197)


julia --project --color=yes -e 'using Pkg; Pkg.instantiate()'
julia --project --color=yes -O0 build_local.jl --cc=clang --gcc_host_compiler_path=
working-directory: ${{ env.REACTANT_DIR }}/deps
Expand Down