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

Documentation bug: mxnet.ndarray.contrib.foreach doesn't accept name #19001

Open
DNXie opened this issue Aug 24, 2020 · 2 comments
Open

Documentation bug: mxnet.ndarray.contrib.foreach doesn't accept name #19001

DNXie opened this issue Aug 24, 2020 · 2 comments

Comments

@DNXie
Copy link

DNXie commented Aug 24, 2020

Description

This is a documentation bug. In the document of mxnet.ndarray.contrib.foreach, the Parameter section stated a parameter name. But it is not in the signature and not accepted by the function.

Link to the document: https://mxnet.apache.org/versions/1.6/api/python/docs/api/ndarray/contrib/index.html#mxnet.ndarray.contrib.foreach

image

Error Message

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: foreach() got an unexpected keyword argument 'name'

To Reproduce

import mxnet as mx
step = lambda data, states: (data + states[0], [states[0] * 2])
data = mx.nd.random.uniform(shape=(2, 10))
states = [mx.nd.random.uniform(shape=(10))]
outs, states = mx.nd.contrib.foreach(step, data, states, name='op')

Environment

We recommend using our script for collecting the diagnositc information. Run the following command and paste the outputs below:

curl --retry 10 -s https://raw.githubusercontent.com/dmlc/gluon-nlp/master/tools/diagnose.py | python

# paste outputs here

OS: ubuntu 18.04
Python: 3.7.6
pip: 20.0.2
numpy: 1.18.5
mxnet: 1.6.0

@szha
Copy link
Member

szha commented Aug 24, 2020

@szha szha added good first issue and removed v1.x Targeting v1.x branch labels Aug 24, 2020
ekdnam added a commit to ekdnam/incubator-mxnet that referenced this issue Sep 1, 2020
name is not a parameter. issue: apache#19001
@ekdnam ekdnam mentioned this issue Sep 1, 2020
6 tasks
szha added a commit that referenced this issue Sep 2, 2020
* set_lower_bound(1) so that stride is not zero

in issue: #18942, an error was occurring where strides became zero. to solve the issue, the lower bounds of stride1 and stride2 have been set to 1.

* small typo

'use_unifrom' changed to 'use_uniform'

* add tests to check lower bound

checking that stride1 and stride2 are be greater than zero

* Update test_operator.py

* documentation error

name is not a parameter. issue: #19001

* data is the parameter, not input

solves issue: #19000

* fix docs

threshold is not a parameter. issue: #18999

* fix docs

context 'ctx' is not a parameter. issue: #18990

* fix indentation

issue: #18988

Co-authored-by: Sheng Zha <szha@users.noreply.github.com>
@vijaya-lakshmi-venkatraman

Hello,
I see a PR merged for this issue.
Is this still open?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants