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

numpy_split support param type fix; numpy_vsplit added #15346

Closed
wants to merge 40 commits into from
Closed

numpy_split support param type fix; numpy_vsplit added #15346

wants to merge 40 commits into from

Conversation

zoeygxy
Copy link
Contributor

@zoeygxy zoeygxy commented Jun 25, 2019

Description

Added python list and set as supported types for parameter 'indices_or_sections'
Added python frontend support for numpy.vsplit

Checklist

Essentials

  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage:
  • Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
  • Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
  • Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
  • Code is well-documented:
  • For user-facing API changes, API doc string has been updated.
  • For new C++ functions in header files, their functionalities and arguments are documented.
  • For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
  • To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

  • Added python list and set as supported types for parameter 'indices_or_sections'
  • Added python frontend support for numpy.vsplit

Comments

  • Note: numpy.split and numpy.vsplit still does not support type ndarray for parameter 'indices_or_sections'.

haojin2 and others added 28 commits June 3, 2019 23:10
@haojin2 @reminisce
[Do not review] [Do not merge] New numpy-compatible sum (apache#14739)
ffd690d 
* Add numpy namespace and initial impl of np.sum (not complete)

* Clean up

* Fix import error

* numpy sum

* add test and backward data type support

* add license to test_numpy_op.py

* improve test to reduce flakiness

* fix sanity build

* extra numeric test and imperative test

* add error message for initial argument
@reminisce
[numpy] Infra for supporting numpy ops in imperative mode and Gluon A…
2f71241 
…PIs (apache#14758)

* Infra of new ndarray and symbol types for numpy operators

* Rename

* Fix import problem

* Refactor

* Remove redundant code

* Add docstring

* More on numpy ndarray and symbol

* Override unimplemented methdos for ndarray and _NumpySymbol

* Fix built-in methods of ndarray and _NumpySymbol

* Fix test and sanity check

* Fix pylint

* Address cr comments

* Add unit tests for ndarray and _NumpySymbol

* Add _true_divide

* Fix gpu build

* Add future import division

* More correct way of checking if an output is from a np compat op

* Fix gpu build

* Fix output ndarray/symbol types with at least one new ndarray/symbol

* Modify true_divide doc

* Fix flaky copying zero-size arrays via gpus

* Fix zero size in gluon hybridize and zeros/ones symbol not creating new symbol type

* Fix doc
@reminisce
Enable np op compat check with name prefix (apache#14897)
355254e
@haojin2 @reminisce
[numpy] Numpy dot (apache#14831)
7a12654 
* Numpy Dot case 1-4 + case 3.5 forward and 0.5 backward

* Backward computation and test coverage
@haojin2 @reminisce
numpy-compatible mean (apache#14859)
4325676
@reminisce
[numpy] Some np ops for d2l (apache#14924)
0ee508e 
* Add np transpose

More ops and namespaces for submodules

Add relu and sigmoid

Add reshape

Fix symbolic name mismatch

Add maximum and minimum

* Add convenience fluent method

* Add ndarray.item()

* Fix CI

* Fix lint

* Fix lint

* Fix reshape gpu

* Add example

* Remove python notebook outputs

* Remove notebook output

* Add one more example
@reminisce
[numpy] Refactor np modules (apache#14989)
043f01e 
* Refactor

* Initial refactoring

* Fix notebook

* Move numpy op check from backend to frontend

* Add homogeneous ndarray check

* Fix grouping inhomogeneous types of symbols

* Improve error handling of different types of symbols as outputs

* Fix test

* Fix numpy test

* Fix ci

* Try to fix gpu ci failure
@reminisce
[numpy] Refactor np module (example runs through) (apache#15055)
308bcc6 
* Refactor notebook

* notebook working with hybrid block

* More refactoring

* Remove unnecessary use_np_compat

* Use class decorator to initialize numpy ndarrays in parameter.py

* Clear notebook outputs

* Improve np decorator

* Remove npe op from optimizer

* Fix CI

* Fix functools.wraps issue in Python2

* Fix ci
@reminisce
Change np_compat to np_shape
6e33cc1
@reminisce
Temporarily disable test_amp
877e0e9
@haojin2 @reminisce
Numpy-compatible stack (apache#15027)
9911675 
* numpy stack

* migrate to use_np_shape
@haojin2 @reminisce
Numpy Unary Ops (apache#15010)
ac92ec3 
* Unary Ops

* new version of unit tests
@reminisce
[numpy] Fix np branch after rebase (apache#15086)
0010c43 
* Add np_array semantics for Gluon

Fix notebook

Fix sanity

Fix gluon deferred infer shape

Add np.random.uniform

Add random normal

Add boolean comparison ops

Add np.ndarray indexing

Reformat test ndarray indexing

Fix unit tests

Add one more test of indexing

Fix sanity

Enable amp test

Add np.arange

Revert cython unit test to ctypes

Delete unnecessary use_np_shape decorator from test

Rebase with numpy branch

support range as index

Fix python2 range type check

Add argmax

Disable clojure test

* Fix ci

* Add np.linalg.norm for ord='fro'

* Fix pylint
@haojin2 @reminisce
numpy concatenate (apache#15104)
aa5153f
@reminisce
[WIP][numpy] Fix for D2L Chapters 2/3/4 (apache#15139)
517451b 
* Fix

* Fix linear regression gluon

* More fix

* Fix pylint

* Fix for chapter 4

* Add np.add mul div mod pow sub and shuffle

* Fix model selection, underfitting, overfitting

* Fix weight decay

* Fix dropout

* Fix

* Fix chapter 4
@reminisce
[numpy] Fix d2l performance regression (apache#15173)
df47457 
* Add np array adapter decorator for layers

* Fix performance regression caused by too many conversions between nd.NDArray and np.ndarray

* Fix pylint

* Fix test backward compatibility issue

* Fix test_lambda
@reminisce
Fix (apache#15188)
4e1274f
@haojin2
fix for chapter6 conv nn (apache#15224)
4ad56dc
@reminisce
[numpy] Fix d2l chapter8 (apache#15237)
fb10e28 
* Add np op doc

* Fix several issues

* Add a N-D dot b 2D support

* Simplify array creation api

* Add swapaxes

* Fix rnn gluon

* More fix

* Fix pylint

* Delete

* Fix mp windows
@haojin2 @reminisce
fix for ch11 (apache#15244)
a00a2ce
@haojin2 @reminisce
Numpy-compatible split (apache#15049)
93679fe 
* numpy split

* numpy split

* unit test

* unit test
@reminisce
[numpy] [DO NOT MERGE] Fix d2l chapters 9 and 13 (apache#15246)
bf4dc33 
* Add npx batch_dot and topk

* Text embedding uses numpy

* Fix SoftmaxCrossEntropyLoss with np

* Fix sentiment cnn

* Fix pylint

* Fix dot attention

* Fix seq2seq attention

* Add np.tile

* Fix transformer

* Fix ci

* Fix ci and rebase
@reminisce
[numpy] Fix d2l chapter 5 (apache#15264)
96520cb 
* Fix parameter initializer

* Add np.save and np.load

* Fix read-write

* Fix lint
@stu1130 @reminisce
Numpy compatible max (apache#15161)
3409578 
* numpy amax

* weird cu file diff

* fix the unit test error

* fix gpu bug

* minor fix

* fix lint

* remove scalar value check

* fix the bug on unit test

* fix the case () that breaks the kernel launch

* add zero dimension unit test

* revert the tuple change

* use mshadow maximum

* remove test zero

* change the macro for now

* change the cuda to use mashadow op

* fix the broadcast_reduce_op_value.cu wrong kernel

* add more logic in shape to detect the invalid situation

* change back to type swtich

* change to as_nd_ndarray

* add missing @npx.use_np_shape

* retrigger CI

* address the comment

* undo algorithm import

* remove the numeric gradient check
@stu1130 @haojin2
Numpy compatible multinomial (apache#15219)
1b62426 
* draft of multinomial

* rename to more concise name

* finish shape

* complete the forward function

* complete forward without handle 0 dimension & scalar

* handle 0 dimension

* add new line

* fix lint

* fix the build error

* fix lint

* finish unit test

* change the registration

* make multinomial support pvals as mx.ndarray

* delete newline

* fix lint error

* support input as list, mx.ndarray, np.ndarray & unit test

* fix lint

* fix the include error

* fix lint

* refactor & pass the tensor instead of tuple to kernel

* fix lint

* updata the doc

* address the comment
@stu1130 @reminisce
Numpy compatible linspace (apache#15256)
2260667 
* draft

* finish linspace implementation

* finish linspace

* delete newline

* fix pylint

* add more unit test

* address comment

* add more test case

* disable too-many-arguments

* resolve confliction

* add ctx
@haojin2 @reminisce
numpy-compatible cumsum (apache#15309)
f803dde
@reminisce
[numpy] Misc fix for other chapters (apache#15332)
a563b56 
* Add np.prod

* Fix ndarray.reshape accepting positional integers as arguments

* Rebase

* Fix rebase error

* Add np.ndarray.flatten

* Fix

* Add broadcast_to

* Add meshgrid and broadcast_arrays

* Fix sin, cos, sinh, cosh not supporting scalars

* Add more unary ops supporting python scalars

* Fix

* Fix

* Fix ci

* Fix sanity
@zoeygxy zoeygxy requested a review from szha as a code owner June 25, 2019 02:36
@zoeygxy
Copy link
Contributor Author

zoeygxy commented Jun 25, 2019

@haojin2 Please review the changes. Thanks!

@reminisce
[numpy] Change d2l chapters cv and gan to use numpy (apache#15368)
6b7525c 
* Change op name style to lower case underscore

* Add ops under image to npx

* Add image submodule to npx

* Fix split_and_load use np

* Fix fine tuning

* Fix bbox and anchor

* Fix odd

* Fix ssd and rcnn

* Remove restriction on binary element-wise scalar

* Fix gan

* Fix sanity

* Try to fix website build failure

* Add npx.random.seed

* Fix doc
@haojin2 haojin2 added the Numpy label Jun 27, 2019
@zoeygxy
Copy link
Contributor Author

zoeygxy commented Jul 2, 2019

Documentation updated.

@haojin2 haojin2 force-pushed the numpy branch 4 times, most recently from a23120e to 07d7a94 Compare July 24, 2019 18:59
@zoeygxy
Copy link
Contributor Author

zoeygxy commented Aug 23, 2019

This PR will be directly merged to master branch. (#15983)

@zoeygxy zoeygxy closed this Aug 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@haojin2 haojin2 haojin2 left review comments

@szha szha Awaiting requested review from szha

Assignees
No one assigned
Labels
Numpy
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@zoeygxy @haojin2 @reminisce @stu1130 @hzfan