Skip to content
This repository has been archived by the owner on Aug 11, 2020. It is now read-only.

Shall we add half_t support to Random<cpu, DType> #306

Open
KiddoZhu opened this issue Nov 6, 2017 · 2 comments
Open

Shall we add half_t support to Random<cpu, DType> #306

KiddoZhu opened this issue Nov 6, 2017 · 2 comments

Comments

@KiddoZhu
Copy link

KiddoZhu commented Nov 6, 2017

Member functions in Random<cpu, DType> are not overload for half_t type. As a result, it fails when we use Random<cpu, half_t> to generate a tensor. This is quite inconvenient for generating noise inside a mxnet operator that is templated by DType.

Shall we add half_t support to Random<cpu, DType>?

@BiranLi
Copy link

BiranLi commented Nov 30, 2017

I think Random<gpu, DType> is important too.
It's not supported by curand function in CUDA9. But I support U.

@szha
Copy link
Member

szha commented Aug 4, 2019

This code base has been donated to the Apache MXNet project per #373, and repo is deprecated. Future development and issue tracking should continue in Apache MXNet.

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

No branches or pull requests

3 participants