-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[flashlight-sequence] New port #31914
Conversation
Note: I will be converting your PR to draft status. The above suggested changes are only recommendations. If you are willing to adopt them, please click "ready for review" after making the modifications. If you do not wish to make any changes, please click "ready for review" directly. That way, I can be aware that you've responded since you can't modify the tags. |
Co-authored-by: JonLiu1993 <63675417+JonLiu1993@users.noreply.github.com>
Tested all features successfully in the following triplet:
|
Summary: Pending microsoft/vcpkg#31914 being merged. Pull Request resolved: #15 Test Plan: visual inspection Reviewed By: EntilZha Differential Revision: D46603380 Pulled By: jacobkahn fbshipit-source-id: 7b9065822b79e5fd02b0051fc6940e45111d3791
Adds a port for Flashlight Sequence (https://github.com/flashlight/sequence). Features include:
cuda
-- build CUDA kernels with the packageopenmp
-- build with OpenMP supportTested ports on all platforms/features except for
x64-windows
(with CUDA), but we have a CI baseline for this -- https://app.circleci.com/pipelines/github/flashlight/sequence/65/workflows/3af72357-b9a7-4d66-aa58-5fa65892243b/jobs/526find_package
calls are REQUIRED, are satisfied byvcpkg.json
's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxxvcpkg.json
matches what upstream says.vcpkg.json
matches what upstream says../vcpkg x-add-version --all
and committing the result.