Skip to content

Conversation

giordano
Copy link
Member

No description provided.

@giordano giordano added the github_actions Pull requests that update GitHub Actions code label Jul 26, 2025
@giordano giordano force-pushed the mg/new-runners branch 9 times, most recently from 7bf41ee to 3fd1ce1 Compare July 26, 2025 03:33
@giordano
Copy link
Member Author

@wsmoses the setup is ready from my point of view, the only problem is that the greatly reduced parallelism would actually make things slower overall in case of multiple concurrent jobs.

@giordano giordano changed the title [CI] Use new runners [CI] Prepare setup for new runners Jul 26, 2025
@giordano
Copy link
Member Author

Ok, this is ready. New runners aren't currently used, but the setup has been slightly adapted to work with them, so they can just be added later.

end

length(counts) < length(size) && continue

@testset "inner repeat" begin
@test (@jit repeat(Reactant.to_rarray(x); inner=counts)) ==
@test (@jit repeat(Reactant.to_rarray(x); inner=counts))
Copy link
Contributor

Choose a reason for hiding this comment

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

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
@test (@jit repeat(Reactant.to_rarray(x); inner=counts))
@test (@jit repeat(Reactant.to_rarray(x); inner=counts))

@wsmoses wsmoses merged commit 5b0df2c into main Jul 28, 2025
52 of 58 checks passed
@wsmoses wsmoses deleted the mg/new-runners branch July 28, 2025 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants