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

[Numpy] around #15822

Closed
wants to merge 1 commit into from
Closed

[Numpy] around #15822

wants to merge 1 commit into from

Conversation

tingying2020
Copy link
Contributor

numpy operator around

@tingying2020 tingying2020 requested a review from szha as a code owner August 9, 2019 08:06
@yzhliu yzhliu added the Numpy label Aug 9, 2019
* change the name of argument

* add doc in three files and fix some bug

* change the data type in .h and add test function

    cancel optimization when abs(temp) < 0.5
    modify test on cpu and add test on gpu
    do not support float16
    edit testcase on gpu and add 'Do not support float16 on doc'

* edit doc: support scalar

* adjust the format

* add license

* fix format error

* delete gpu test
@@ -0,0 +1,69 @@
/*
Copy link
Contributor

Choose a reason for hiding this comment

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

put this op in src/operator/numpy/np_elemwise_unary_op_basic files.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Create a new branch for PR.

#print("x = ", x, " d = ", d, "\n")
np_out = _np.around(x.asnumpy(), d)
mx_out = test_around(x)
#print("mx_out = ", mx_out, " np_out = ", np_out, "\n")
Copy link
Contributor

Choose a reason for hiding this comment

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

Get rid of unused code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

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.

None yet

3 participants