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

Springify (modded Spring layout with CUDA kernel) #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Cvikli
Copy link

@Cvikli Cvikli commented May 15, 2022

I find Spring layout sometimes too overcrowded at specified graphs. This layout algorithms sometimes gives a little bit better graph structure.
It uses CUDA for ordering as I need larger graphs sometime to order.

I know there are some works need to be done to be merged. But I hope it is a good start.

@hexaeder
Copy link
Collaborator

I did not have a closer look at the code yet. However, this adds CUDA as a dependency which is quite hefty and not usable by all people. I'd suggest, that a GPU accelerated layouts should live in a separate package like NetworkLayoutCUDA.jl and uses the interface for layouts...

@SimonDanisch
Copy link
Member

Could this also be done with just GPUArrays?

@Cvikli
Copy link
Author

Cvikli commented May 31, 2022

@hexaeder you are right. CUDA sounds a little bit too big dependency. :( I don't see any other option than creating another package for it indeed.

@SimonDanisch Isn't that package somewhat deprecated... or unadvised to be used? Also if it is advised, would it solve our dependency issue? Also shouldn't we use it with kernelabstraction if so?

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

Successfully merging this pull request may close these issues.

3 participants