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

Move TShape definition and necessary passes out of NNVM #14266

Closed
reminisce opened this issue Feb 27, 2019 · 3 comments
Closed

Move TShape definition and necessary passes out of NNVM #14266

reminisce opened this issue Feb 27, 2019 · 3 comments
Labels
Numpy

Comments

@reminisce
Copy link
Contributor

MXNet needs its own definition of TShape to support scalars and zero-size tensors which is compatible with NumPy. To keep NNVM code unchanged, we need to copy tuple.h from NNVM to MXNet and make changes in the current MXNet codebase accordingly. Some NNVM passes need to be moved out too.

See #14253 for more details.
@mxnet-label-bot
Copy link
Contributor

Hey, this is the MXNet Label Bot.
Thank you for submitting the issue! I will try and suggest some labels so that the appropriate MXNet community members can help resolve it.
Here are my recommended labels: Feature

@reminisce
Copy link
Contributor Author

@junrushao1994

@junrushao
Copy link
Member

Working on this #14270

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Numpy
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@reminisce @junrushao @mxnet-label-bot