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

Add Gather_nd Scatter_nd to NDArray API category doc#15689

Merged
aaronmarkham merged 2 commits intoapache:masterfrom
ChaiBapchya:index_routine_op_doc_fix
Aug 6, 2019
Merged

Add Gather_nd Scatter_nd to NDArray API category doc#15689
aaronmarkham merged 2 commits intoapache:masterfrom
ChaiBapchya:index_routine_op_doc_fix

Conversation

@ChaiBapchya
Copy link
Contributor

@ChaiBapchya ChaiBapchya commented Jul 29, 2019

Description

gather_nd and scatter_nd - present in Symbol API Indexing routine category - https://mxnet.incubator.apache.org/versions/master/api/python/symbol/symbol.html#indexing-routines

But not present in Indexing routine category in NDArray API

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage:
  • Code is well-documented:
  • To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

  • Add to Doc

@ChaiBapchya ChaiBapchya requested a review from szha as a code owner July 29, 2019 23:12
@ChaiBapchya
Copy link
Contributor Author

@mxnet-label-bot add [Doc]

@marcoabreu marcoabreu added the Doc label Jul 29, 2019
Copy link
Contributor

@aaronmarkham aaronmarkham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also need this update on the beta site's code.

@ChaiBapchya
Copy link
Contributor Author

@aaronmarkham @apeforest good to merge?
I will open a PR on mli/new-docs with this change. But currently facing this issue while trying to sync my fork mli/new-docs#128

@aaronmarkham aaronmarkham merged commit 8c2bef5 into apache:master Aug 6, 2019
@ChaiBapchya ChaiBapchya deleted the index_routine_op_doc_fix branch August 6, 2019 20:09
anirudhacharya pushed a commit to anirudhacharya/mxnet that referenced this pull request Aug 20, 2019
* gather scatter to NDArray doc

* Trigger notification
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants