Commit df5c2e4
ci: Reduce test time by moving compilation off-line (#2089)
<!-- .github/pull_request_template.md -->
## 📌 Description
Download `flashinfer-cubin` and `flashinfer-jit-cache `to avoid
compilation. (Unless the JIT kernel is not in the
`flashinfer-jit-cache`; then it will still JIT compile during test
runtime. We could set `export FLASHINFER_DISABLE_JIT = 1 `to avoid this,
but then it will "skip" a lot of tests that use JIT kernels that aren't
found in `flashinfer-jit-cache`.)
## 🔍 Related Issues
Issue was discussed on slack. "Ideally, we would move that compilation
off-line which would reduce test time & make kernel hang detection much
easier. "
## 🚀 Pull Request Checklist
Thank you for contributing to FlashInfer! Before we review your pull
request, please make sure the following items are complete.
### ✅ Pre-commit Checks
- [x] I have installed `pre-commit` by running `pip install pre-commit`
(or used your preferred method).
- [x] I have installed the hooks with `pre-commit install`.
- [x] I have run the hooks manually with `pre-commit run --all-files`
and fixed any reported issues.
> If you are unsure about how to set up `pre-commit`, see [the
pre-commit documentation](https://pre-commit.com/).
## 🧪 Tests
- [x] Tests have been added or updated as needed.
- [x] All tests are passing (`unittest`, etc.).
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit
* **Chores**
* Improved runtime install flow to detect CUDA, compute an effective JIT
architecture mapping, and install matching precompiled kernel artifacts
plus local package sources; these steps run only outside dry-run mode
and verify installation by showing configuration.
* Simplified build parallelism calculation to a constant division by 8
(with existing safety guards retained).
* **Bug Fixes**
* Missing precompiled kernel artifacts now cause an explicit error/abort
instead of a warning.
<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
---------
Co-authored-by: yzh119 <[email protected]>1 parent 18004a8 commit df5c2e4
File tree
2 files changed
+70
-1
lines changed- scripts
2 files changed
+70
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
| 14 | + | |
| 15 | + | |
15 | 16 | | |
16 | 17 | | |
17 | 18 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
28 | 90 | | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
29 | 97 | | |
30 | 98 | | |
31 | 99 | | |
| |||
0 commit comments