Skip to content
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

restore benchmarking for dp layers and fix nightly tests #508

Closed
wants to merge 5 commits into from

Conversation

ffuuugor
Copy link
Contributor

@ffuuugor ffuuugor commented Sep 14, 2022

Ok, this time it should work.

See successful CircleCI run here. Normally we don't run benchmarks for every commit to save time. So here I temporarily added benchmarks to commit tests, made sure they pass and then disabled them back. CircleCI link above include previously failing microbenchmarks

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 14, 2022
@facebook-github-bot
Copy link
Contributor

@ffuuugor has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

@ffuuugor has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@ffuuugor ffuuugor marked this pull request as ready for review September 14, 2022 13:59
@ffuuugor ffuuugor changed the title restore benchmarking for dp layers restore benchmarking for dp layers and fix nightly tests Sep 14, 2022
Copy link
Contributor

@alexandresablayrolles alexandresablayrolles left a comment

Choose a reason for hiding this comment

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

LGTM

@karthikprasad karthikprasad deleted the ffuuugor_benchmark_fix2 branch September 27, 2022 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants