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

Breeze fixes #11537

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Breeze fixes #11537

wants to merge 7 commits into from

Conversation

dreveman
Copy link

No description provided.

@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 Nov 14, 2024
Copy link

netlify bot commented Nov 14, 2024

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 3eca5ae
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/673668ca9bddf000083577f1

@dreveman
Copy link
Author

@Yuhta I found these issues when trying to build breeze tests in ToT. The cmake change is just a result of my MR falling behind before it was merged and the other issues turned up when using ubuntu 24.04 instead of 22.04 to build.

@Yuhta Yuhta added the ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall label Nov 14, 2024
@Yuhta Yuhta mentioned this pull request Nov 14, 2024
@dreveman
Copy link
Author

@Yuhta Added a commit that provides an autogen header for generated files as discussed

@Yuhta
Copy link
Contributor

Yuhta commented Nov 14, 2024

@dreveman
Copy link
Author

@dreveman Something to be fixed with formatter: https://github.com/facebookincubator/velox/actions/runs/11843156393/job/33008280565

Yeah, I'll take care of that and also include a number of changes that are needed to build perf tests and tests for other platforms after everything was moved into the breeze subdir

David Reveman added 2 commits November 14, 2024 16:16
And for unit tests when generating kernel code at build time
instead of using checked in code.
@dreveman
Copy link
Author

@dreveman Something to be fixed with formatter: https://github.com/facebookincubator/velox/actions/runs/11843156393/job/33008280565

Yeah, I'll take care of that and also include a number of changes that are needed to build perf tests and tests for other platforms after everything was moved into the breeze subdir

@Yuhta Should be fixed now and this PR should be ready to be merged

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. ready-to-merge PR that have been reviewed and are ready for merging. PRs with this tag notify the Velox Meta oncall
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants