Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge refactored collenchyma code #2

Closed
jonysy opened this issue Jan 31, 2017 · 0 comments
Closed

Merge refactored collenchyma code #2

jonysy opened this issue Jan 31, 2017 · 0 comments

Comments

@jonysy
Copy link
Owner

jonysy commented Jan 31, 2017

Useful

decoupling branch
Backend decoupling and typed memory interface #63
collenchyma-cuda-refactor
alexandermorozov/collenchyma
Refactor synchronization #62 PR
simplify syncing

Gitter channel:

alexander: @\matrixbot I think f32 should be enough. Nvidia has recently introduced support for f16 in Pascal GPUs, so f16 is probably enough too most of the time

alexander: @\jonysy collenchyma-blas supports some simple operations... For A * 3 there is Scal trait with scal() function on backend

jonysy: Hello @\alexandermorozov. I’m glad you joined. Would you mind if I merge your PR with my hard fork of Collenchyma?

alexander: @\jonysy Of course merge it, I'll be happy that the code is used. You can also cherry-pick from my repo, it may be more convinient. That said, most of my currently opened PRs add autosync to SharedTensor, and in this project I think it would be better to have explicit sync as graph node

jonysy: @\alexandermorozov Great!

jonysy: Unfortunately, there’s currently no support for BLAS on OpenCL using Rust 😕.
Is there an alternative approach I can take?

cathal: Actively learning Arrayfire right now with that goal in mind.

alexander: @\jonysy You can use collenchyma with "native" backend, but that's not fun ) There was a rust project that aimed to support NNs on openCL, and it afaik it had some blas functions implemented

alexander: @\jonysy deeplearn-rs

jonysy: Hmm. I couldn’t locate any blas functions..
Looks like everything has been moved over to https://github.com/tedsta/gpuarray-rs
Thanks

cathal: I asked him about gpuarray-rs (see the issues pane) and he feels it's immature and needs a major refactor to be more composable/maintainable. IIRC, he was open to participation.

@jonysy jonysy changed the title Merge Alexander's PR Merge refactored collenchyma code Jan 31, 2017
@jonysy jonysy closed this as completed in 31a2547 Mar 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant