Skip to content
This repository has been 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
merged 2 commits into from
Aug 6, 2019

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