-
Notifications
You must be signed in to change notification settings - Fork 444
UCCL: Simplify and Optimize for batch transfers #1271
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
Merged
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
90ed9db
Add support to prepare Fifo
praveingk 8a354ea
Fix errors
praveingk a9bcd2b
Add vector read/write support
praveingk 8fd6a22
Use prepare fifo APIs
praveingk 897f437
Remove RCMODE
praveingk 5921a68
Remove rcmode in prepXfer
praveingk f03ee88
Remove rcmode from CI
praveingk 76f8c81
Fix cleanup
praveingk e80c85b
Maintain a single transfer ID
praveingk 047fd3c
Fix formatting
praveingk 0438400
Fix formatting after changes
praveingk 83a8eea
Remove redundant FIFO_ITEM_SIZE
praveingk d70747d
Update commit SHA of UCCL
praveingk 92f0466
Remove unused variable
praveingk d276d72
Remove unused vars
praveingk 800179b
Fix error
praveingk be118d1
Add recent updates to README
praveingk 2f94415
Merge branch 'main' into one_sided_async
praveingk dc3d822
Fix copyright and format
praveingk c24e886
Merge branch 'one_sided_async' of github.com:praveingk/nixl into one_…
praveingk 90d27cb
Increase CI image tag
praveingk facdff6
Use latest UCCL commit SHA
praveingk dbc0a46
Merge branch 'main' into one_sided_async
brminich 8bcdc5a
Merge branch 'main' into one_sided_async
brminich 3f081c2
Merge branch 'main' into one_sided_async
brminich 1c9ae62
Merge branch 'main' into one_sided_async
brminich 5b2fa0b
Merge branch 'main' into one_sided_async
brminich 67aa7eb
Merge branch 'main' into one_sided_async
praveingk abda8a1
Merge branch 'main' into one_sided_async
brminich 99df3d5
Add recent UCCL Commit
praveingk d956a79
Merge branch 'one_sided_async' of github.com:praveingk/nixl into one_…
praveingk 8a05015
Merge branch 'main' into one_sided_async
praveingk b8fd08e
Merge branch 'main' into one_sided_async
praveingk d1e6536
Remove sleep during cleanup
praveingk ca02d54
Merge branch 'main' into one_sided_async
praveingk 3144cf9
Add stop_accept API for graceful cleanup
praveingk 867990b
Merge branch 'main' into one_sided_async
praveingk 1bc2d51
Use CodeRabbit suggestions
praveingk 1deed0d
Merge branch 'one_sided_async' of github.com:praveingk/nixl into one_…
praveingk 7971222
Merge branch 'main' into one_sided_async
praveingk ca40dee
Add cleanup change to address crash
praveingk ea89faf
Merge branch 'one_sided_async' of github.com:praveingk/nixl into one_…
praveingk 94e0564
Merge branch 'main' into one_sided_async
praveingk 9cff793
Bump up the IMAGE_TAG
praveingk bb7b0a7
Revert CI_IMAGE_TAG
praveingk 020ed23
Merge branch 'main' into one_sided_async
praveingk 872e33a
Update IMAGE_TAG
praveingk 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
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.
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.
Add a trailing newline to satisfy markdownlint (MD047).
📝 Suggested fix
📝 Committable suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 51-51: Files should end with a single newline character
(MD047, single-trailing-newline)
🤖 Prompt for AI Agents