Skip to content

Move PR Continuous Integration to Github Actions #8534

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

Jump to bottom
Merged
merged 22 commits into from
Apr 23, 2025
Merged

Move PR Continuous Integration to Github Actions #8534

merged 22 commits into from
Apr 23, 2025

Conversation

mattsoulanille
Copy link
Member

@mattsoulanille mattsoulanille commented Mar 27, 2025 •
edited
Loading

Github Actions have a MacOS runner, which lets us test WebGPU.

Example run: https://github.com/mattsoulanille/tfjs/actions/runs/14521816238

@mattsoulanille
Basic CI workflow to test building tfjs
d68e24a
@mattsoulanille
Add manual trigger
dd24f37
@mattsoulanille
Make action test tfjs
78f634c
@mattsoulanille
Test GPU separately from CPU
868b6a5
@mattsoulanille
Fix typo
91121bd
@mattsoulanille
Debugging tests: just one GPU test
021f408
@mattsoulanille
Test karma on mac
cb69276
@mattsoulanille
Simple browser test on web mac
fbf4f95
@mattsoulanille
linux vs mac
90377b4
@mattsoulanille
Test GPU on linux
094ec6d
@mattsoulanille
With bazel cache
43c4f22
@mattsoulanille
Use non-headless for webgpu
7b06222
@mattsoulanille
Use Chrome instead of ChromeCanary
0664128
@mattsoulanille
ChromeHeadless for linux
fc18945
@mattsoulanille
Use the same browser for WebGPU and WebGL tests
9fa06f4
@mattsoulanille
WebGPU skip cos, sin nan propagation tests
682dd84
@mattsoulanille
Disable linux gpu tests
67502d3
@mattsoulanille
Skip pow int32^int32 returns int32 test
6dbe8eb
@mattsoulanille
Fix pow exclusion
d17f7a2
@mattsoulanille
Disable MultiHeadAttention test
3e6c66a
@mattsoulanille
Fix incorrect pow int32 exclusion from tests
4e4cea5
@mattsoulanille
Remove commented lines in workflow
f81dd34
@mattsoulanille mattsoulanille changed the title WIP Github Actions Move PR Continuous Integration to Github Actions Apr 17, 2025
@mattsoulanille mattsoulanille marked this pull request as ready for review April 17, 2025 18:18
@mattsoulanille mattsoulanille requested a review from pyu10055 April 17, 2025 18:18
pyu10055
pyu10055 approved these changes Apr 21, 2025
View reviewed changes
Copy link
Collaborator

@pyu10055 pyu10055 left a comment

Choose a reason for hiding this comment

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

@mattsoulanille thank you for making the switch.

@mattsoulanille mattsoulanille merged commit 78345c5 into tensorflow:master Apr 23, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pyu10055 pyu10055 pyu10055 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mattsoulanille @pyu10055