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.

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

Add 3.13t CI #256

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Add 3.13t CI #256

wants to merge 4 commits into from

Conversation

mthrok
Copy link
Collaborator

@mthrok mthrok commented Oct 23, 2024

No description provided.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 23, 2024
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@mthrok mthrok force-pushed the ft3 branch 11 times, most recently from d085c47 to f7e176e Compare March 5, 2025 22:30
@mthrok mthrok force-pushed the ft3 branch 5 times, most recently from 36ca83e to 2e0da9b Compare March 9, 2025 20:57
@mthrok mthrok marked this pull request as draft March 9, 2025 20:59
@mthrok mthrok force-pushed the ft3 branch 5 times, most recently from 48f1c6d to d2568e1 Compare March 9, 2025 22:11
@mthrok mthrok changed the title Update build for FT Python Add 3.13t CI Mar 10, 2025
@mthrok mthrok force-pushed the ft3 branch 5 times, most recently from ea1b9df to 78bb5f6 Compare March 12, 2025 01:56
@mthrok
Copy link
Collaborator Author

mthrok commented Mar 12, 2025

Strangely, uv with 3.13t builds 3.12 extension.

https://github.com/facebookresearch/spdl/actions/runs/13802000565/job/38605981442?pr=256#step:4:228-246

+ uv python list --only-installed
cpython-3.13.2+freethreaded-linux-x86_64-gnu    /opt/python/cp313-cp313t/bin/python -> python3
...
+ v=3.13
+ [[ ft == \f\t ]]
+ dir=cp313-cp313t
+ python_exe=/opt/python/cp313-cp313t/bin/python
+ uv venv --python /opt/python/cp313-cp313t/bin/python
Using CPython 3.13.2 interpreter at: /opt/python/cp313-cp313t/bin/python
Creating virtual environment at: .venv
...
creating build/lib.linux-x86_64-cpython-312/spdl/io
                                        ^^^
...
Running "cmake -S /__w/spdl/spdl/packaging/spdl_io/src/third_party -B build/temp.linux-x86_64-cpython-312/deps_first_stage -DCMAKE_VERBOSE_MAKEFILE=OFF -DCMAKE_INSTALL_MESSAGE=NEVER -DCMAKE_INSTALL_PREFIX=build/temp.linux-x86_64-cpython-312 -GNinja"
...

mthrok added 4 commits March 18, 2025 00:18
@mthrok
Move build query functions
8c6dbed
@mthrok
Run tests on NVDEC build
5b71651
@mthrok
Test CPU with CUDA package
3fa43af 
Now that we split the CUDA / core features, CUDA packages
can be used in machines without GPU.

This commit merges the build for x86 into CUDA/NVDEC and run
unittest on machines with both w/ and w/o GPUs.
@mthrok
Add 3.13t build CI jobs for Linux
ea33f0a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mthrok @facebook-github-bot