This repository has been archived by the owner on Nov 17, 2023. It is now read-only.
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.
[MXNET-#16795] Byteps-KVStore: Intergrate Byteps into mxnet as new type of kvstore backend #17555
[MXNET-#16795] Byteps-KVStore: Intergrate Byteps into mxnet as new type of kvstore backend #17555
Changes from 22 commits
011f1bb
017a225
933c8c2
3a5c78a
f183b50
95c119b
d02da35
2bf7bf6
7bfeb33
dcc003b
03c505f
f5f0f6d
e244561
2dbabb6
cdb6d2e
1901919
9223b2a
b54c215
f359299
1a2b269
842649e
38917ee
9a4e639
7435a8c
1871071
bad53a0
0951911
b55001c
a4ec4b8
55adf68
d024ca9
46bfd8d
a9a13b0
6d48642
73ffc37
97497bc
0177bf5
5f5fa85
db173ba
13e917c
01eeab3
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
The broadcast API is generic and users don't expect the value of
value
to be changed. We can optimize for the common case, where users callkv.broadcast('x', value=x, out=x)
orkv.broadcast('x', value=x, out=[x])
. This is usually used for initializing all network parameters.For this case we can check the value of
inplace
, if so we modify value inplace and callbyteps_push_pull
. Otherwise, we need to create use temporary buffers to perform pushpull.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.
Users don't expect
value
to change unlessout is value