-
Notifications
You must be signed in to change notification settings - Fork 6.8k
min fix of 'storage_type' index mismatch #14560
Conversation
Index of array 'storage_type' should be indice of 'node entries', not 'nodes'. sign-of-by: [email protected]
Thank you for the fix @gssplayer Could you have a look at the CI failure? @mxnet-label-bot add [Backend, pr-awaiting-review] |
@gssplayer Can you look into the CI failures in order to take this PR forward ? |
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.
Thanks for the fix. LGTM.
Please rebase and retrigger CI. |
@@ -947,8 +947,8 @@ void GraphExecutor::InitDataEntryMemory(std::vector<NDArray>* shared_pool) { | |||
for (uint32_t i = 0; i < idx[nid].source->num_outputs(); ++i) { | |||
auto eid = idx.entry_id(nid, i); | |||
data_context[eid] = vctx[nid]; | |||
CHECK_NE(vstorage_type[nid], kUndefinedStorage); | |||
data_storage_type[eid] = (NDArrayStorageType) vstorage_type[nid]; |
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.
Thanks for the fix! Is there any failed unit test you have at hand?
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.
I'am very sorry for no response on this question. I found this problem just by reading code, and had no any test. I've no runtime enviroment at hand. And I have to admit your CI is too complicate for me. Can anyone help to continue tracking this problem?
@gssplayer Thanks for the contribution, could you rebase your PR and trigger CI test again? |
@gssplayer Could you have a look at the CI failure? You could try re-triggering it. |
@mxnet-label-bot update [Backend, pr-awaiting-response] |
Triggered. hopefully it passes |
@gssplayer The CI passed :) @eric-haibin-lin Can this PR be merged if it looks good ? |
Index of array 'storage_type' should be indice of 'node entries', not 'nodes'. sign-of-by: [email protected]
Index of array 'storage_type' should be indice of 'node entries',
not 'nodes'.
sign-of-by: [email protected]
Description
(Brief description on what this PR is about)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
Comments