-
Notifications
You must be signed in to change notification settings - Fork 191
Run CI on Modal, upgrade Bitsandbytes #641
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
Merged
Changes from 54 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
fc52696
Run CI on Modal, upgrade Bitsandbytes
mryab 58f3d44
Add docs configuration
mryab 6d36cd1
Fix formatting
mryab ab714bd
Configure concurrency for Modal tests
mryab c840ab9
Sort imports
mryab f717bf6
Set up the timeout
mryab 0dca5a2
Set up concurrency for other actions as well
mryab 11feccf
Remove concurrency limits
mryab cbf4450
Add concurrency, update bitsandbytes in dependencies
mryab 4f303bd
Add cache, bump CI versions
mryab 6a5ec5e
Skip test_allreduce_protocol for the time being
mryab ba3e386
Reduce the number of CPUs
mryab 1fb8dec
Decrease the limits in test_dht_connection_successful
mryab 67e040f
Restore the limits in test_dht_connection_successful
mryab c0af379
Clear the blacklist before attempting store
mryab 6116570
Increase the wait in test_load_state_from_peers
mryab 801bb4f
Parametrize tests by Python version, upload Codecov coverage
mryab fd69b64
Check out and build a specific version of bitsandbytes
mryab 22739f5
Increase the timeouts to account for image builds
mryab 635879f
Introduce timeouts
mryab 8fbd9dd
Increase the number of CPUs for tests
mryab d70b4b9
Make tests more robust
mryab 4254468
Make tests more robust
mryab 1753bae
Reformat the code
mryab 4753fef
Mark test_client_disconnect as flaky
mryab 9705318
Build and test p2pd separately
mryab ae5ed98
Install Go only for a specific image
mryab 11eb277
Don't use uv when building p2pd
mryab 9d37fe9
Mark test_dhtnode_blacklist as flaky
mryab 7abc9f0
Increase timeouts
mryab 5b69835
Make test_averaging_trigger more robust
mryab 9e37679
Download codecov with wget
mryab aa20215
Skip all training tests for the time being
mryab a03288e
Skip test_allgather for the time being
mryab a614a02
Mark test_performance_ema_threadsafe and test_remote_expert_worker_ru…
mryab 2cfc94a
Reduce timeouts, mark test_background_server_identity_path as flaky
mryab df048db
Mention sponsorship by Prime Intellect
mryab e388e07
Fix missing import
mryab 98e6a38
Mark flaky tests
mryab b317b29
Modify the codecov command
mryab 66c9187
Pass extra environment variables to codecov
mryab 93460aa
Remove --dist from codecov run
mryab 75529a1
Pass GITHUB_EVENT_PULL_REQUEST_HEAD_SHA when running the test
mryab 83b53bb
Mark test_fault_tolerance as flaky
mryab 5984bad
Mark test_cli_run_server_identity_path as flaky
mryab 2f67c52
Disable parallel execution for codecov management
mryab e8efb66
Increase codecov run timeout to 15 minutes
mryab f8ad2a8
Pass GITHUB_EVENT_PULL_REQUEST_HEAD_SHA to the workflow
mryab 225439e
Pass additional secrets
mryab 3695813
Mark one more test as flaky
mryab 6bac780
Mark another test as flaky
mryab 3228dfd
Pass codecov values explicitly
mryab 0a9347d
Pass --no-use-pep517 to uv pip install
mryab 87f0ece
Change uv pip to pip
mryab 46fa9f5
Extract the blocksize for quantization into a constant
mryab 717dd34
Fix missing newline
mryab 0fcd2ba
Rewrite test_averaging_trigger with time.monotonic
mryab cfa51d2
Replace os.unlink with os.remove
mryab File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,112 @@ | ||
name: Modal tests | ||
|
||
on: | ||
push: | ||
branches: [master] | ||
pull_request: | ||
|
||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
cancel-in-progress: true | ||
|
||
jobs: | ||
run_tests: | ||
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
python-version: ["3.9", "3.10", "3.11", "3.12"] | ||
fail-fast: false | ||
env: | ||
MODAL_TOKEN_ID: ${{ secrets.MODAL_TOKEN_ID }} | ||
MODAL_TOKEN_SECRET: ${{ secrets.MODAL_TOKEN_SECRET }} | ||
PYTHON_VERSION: ${{ matrix.python-version }} | ||
timeout-minutes: 15 | ||
steps: | ||
- name: Checkout Repository | ||
uses: actions/checkout@v4 | ||
|
||
- name: Install Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: "3.12" | ||
|
||
- name: Cache dependencies | ||
uses: actions/cache@v4 | ||
with: | ||
path: ~/.cache/pip | ||
key: Key-v1-3.12-modal | ||
|
||
- name: Install build dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install modal==0.73.32 | ||
|
||
- name: Run tests | ||
run: | | ||
modal run modal_ci.py::run_tests | ||
|
||
measure_coverage: | ||
runs-on: ubuntu-latest | ||
env: | ||
MODAL_TOKEN_ID: ${{ secrets.MODAL_TOKEN_ID }} | ||
MODAL_TOKEN_SECRET: ${{ secrets.MODAL_TOKEN_SECRET }} | ||
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} | ||
GITHUB_EVENT_NAME: ${{ github.event_name }} | ||
GITHUB_EVENT_NUMBER: ${{ github.event.number }} | ||
GITHUB_EVENT_PULL_REQUEST_HEAD_SHA: ${{ github.event.pull_request.head.sha }} | ||
PYTHON_VERSION: "3.11" | ||
timeout-minutes: 15 | ||
steps: | ||
- name: Checkout Repository | ||
uses: actions/checkout@v4 | ||
|
||
- name: Install Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: "3.12" | ||
|
||
- name: Cache dependencies | ||
uses: actions/cache@v4 | ||
with: | ||
path: ~/.cache/pip | ||
key: Key-v1-3.12-modal | ||
|
||
- name: Install build dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install modal==0.73.32 | ||
|
||
- name: Measure and upload coverage | ||
run: | | ||
modal run modal_ci.py::run_codecov | ||
|
||
build_and_test_p2pd: | ||
runs-on: ubuntu-latest | ||
env: | ||
MODAL_TOKEN_ID: ${{ secrets.MODAL_TOKEN_ID }} | ||
MODAL_TOKEN_SECRET: ${{ secrets.MODAL_TOKEN_SECRET }} | ||
PYTHON_VERSION: "3.11" | ||
timeout-minutes: 10 | ||
steps: | ||
- name: Checkout Repository | ||
uses: actions/checkout@v4 | ||
|
||
- name: Install Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: "3.12" | ||
|
||
- name: Cache dependencies | ||
uses: actions/cache@v4 | ||
with: | ||
path: ~/.cache/pip | ||
key: Key-v1-3.12-modal | ||
|
||
- name: Install build dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install modal==0.73.32 | ||
|
||
- name: Run p2pd tests | ||
run: | | ||
modal run modal_ci.py::build_and_test_p2pd |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,6 +2,7 @@ version: 2 | |
|
||
sphinx: | ||
fail_on_warning: true | ||
configuration: docs/conf.py | ||
|
||
python: | ||
install: | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's better to recompute the number of pending_samples, as some samples might have 0 tasks (for example, in
test_call_many
)