Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

we need to update the doc of scatter_nd and gather_nd #11867

Closed
zheng-da opened this issue Jul 24, 2018 · 7 comments
Closed

we need to update the doc of scatter_nd and gather_nd #11867

zheng-da opened this issue Jul 24, 2018 · 7 comments
Labels
Doc Operator Unclear Error/Doc

Comments

@zheng-da
Copy link
Contributor

zheng-da commented Jul 24, 2018 •
edited
Loading

scatter_nd and gather_nd in MXNet are similar to the ones in TensorFlow. But the current documentation is difficult to understand. We need to update the document.
@haojin2
Copy link
Contributor

haojin2 commented Jul 25, 2018

Working on this.

@haojin2 haojin2 mentioned this issue Jul 27, 2018
Merged
5 tasks
@haojin2
Copy link
Contributor

haojin2 commented Jul 27, 2018

@zheng-da Fix is in #11918

@zheng-da zheng-da changed the title we need to update the doc of scatter_nd we need to update the doc of scatter_nd and gather_nd Jul 28, 2018
@sandeep-krishnamurthy
Copy link
Contributor

Resolved by @haojin2

@sandeep-krishnamurthy
Copy link
Contributor

Reopening as we have few more pending updates - @haojin2

@piyushghai
Copy link
Contributor

piyushghai commented Oct 9, 2018 •
edited
Loading

@haojin2 Was there a PR to track the pending updates for resolving the doc issues here ?

@sojiadeshina
Copy link
Contributor

echoing @piyushghai are there still pending updates for this? @sandeep-krishnamurthy @haojin2 can this be closed now?

@ThomasDelteil
Copy link
Contributor

closing for now, please reopen with details of what needs to be updated if necessary, thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Doc Operator Unclear Error/Doc
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@zheng-da @szha @sandeep-krishnamurthy @ThomasDelteil @eric-haibin-lin @piyushghai @sojiadeshina @haojin2