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.
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
[webgl] Fix nightly browser configurations #5687
[webgl] Fix nightly browser configurations #5687
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 1 approvals obtained (waiting on @mattsoulanille)
tfjs-backend-webgl/BUILD.bazel, line 130 at r2 (raw file):
"bs_firefox_mac", "bs_safari_mac", "bs_ios_11",
do we need to test webgl1 on non-ios browsers ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! 1 of 1 approvals obtained (waiting on @mattsoulanille)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: complete! 1 of 1 approvals obtained (waiting on @pyu10055)
tfjs-backend-webgl/BUILD.bazel, line 130 at r2 (raw file):
Previously, pyu10055 (Ping Yu) wrote…
do we need to test webgl1 on non-ios browsers ?
I don't think so. Removed them.
Run the correct browsers in nightly and pass the correct parameters to them to match the original test-ci.sh file. Pin Firefox to 90. Fix square test comparison precision on Safari webgl1.
Run the correct browsers in nightly and pass the correct parameters to them to match the original test-ci.sh file.
Pin Firefox to 90.
Fix
square
test comparison precision on Safari webgl1.To see the logs from the Cloud Build CI, please join either our discussion or announcement mailing list.
This change is