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.

Sign up for GitHub

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

Jump to bottom

GPU support for ComputeFramework #32

Closed
wants to merge 1 commit into from
Closed

GPU support for ComputeFramework #32

wants to merge 1 commit into from

Conversation

ranjanan
Copy link
Collaborator

Add GPU support using ArrayFire

@shashi
Copy link
Collaborator

shashi commented Apr 25, 2016

What do you think we should do about the ArrayFire dependency? probably @require?

can you create a branch on this repo? I can then add my commits

@ranjanan
Copy link
Collaborator Author

Yeah, @require seems like it's worth trying out. Shall I delete this pull request and create a branch on the repo, which we can then collab on?

@shashi
Copy link
Collaborator

shashi commented Apr 25, 2016

yea sounds good

@ranjanan ranjanan closed this Apr 25, 2016
@ViralBShah
Copy link
Member

ViralBShah commented Apr 25, 2016 •
edited
Loading

Doesn't @require break precompilation.

@shashi
Copy link
Collaborator

shashi commented Apr 25, 2016

On 0.4 at least if you don't have precompile directive, it does work.

The alternative is having users compile ArrayFire to get CF to work, or have a package for interop. Another case for JuliaLang/julia#15705

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

Successfully merging this pull request may close these issues.
3 participants
@ranjanan @shashi @ViralBShah