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

Clean up dependencies #79

Open
5 of 11 tasks
michakraus opened this issue Sep 21, 2023 · 4 comments
Open
5 of 11 tasks

Clean up dependencies #79

michakraus opened this issue Sep 21, 2023 · 4 comments

Comments

@michakraus
Copy link
Member

michakraus commented Sep 21, 2023 •
edited by benedict-96
Loading

Currently we rely on many dependencies which are not really needed, including:

  • GPUArrays
  • Lux
  • NLsolve
  • Optimisers

Others are only needed in Tests, including:

  • ChainRulesTestUtils
  • SafeTestsets

For some packages it is unclear if we still need them, including:

  • Distances
  • ForwardDiff
  • InteractiveUtils
  • SparseArrays
  • StatsBase

All of these need to be verified and removed if possible.
@benedict-96
Copy link
Collaborator

benedict-96 commented Nov 23, 2023 •
edited
Loading

We probably also don't need CUDA. All of the GPU code should be written with KernelAbstractions (which also doesn't have CUDA in its dependencies).

  • CUDA

@benedict-96
Copy link
Collaborator

We have a Lux dependency because of backends/lux.jl. Do we really need this?

@benedict-96
Copy link
Collaborator

Is ForwardDiff needed in anything Theo wrote, for example for HNNs?

@benedict-96
Copy link
Collaborator

#121 gets rid of (i) CUDA, (ii) GPUArrays, (iv) ForwardDiff and (v) Lux.

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

No branches or pull requests
2 participants
@michakraus @benedict-96