Skip to content

Commit 70a773c

Browse files
Update TODO.md
1 parent 9dddb0e commit 70a773c

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

TODO.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ A checklist of planned tasks, improvements, and ideas for the package. Feel free
3030
- [ ] Improve the efficiency of the code where possible. See the general [Julia performance tips](https://docs.julialang.org/en/v1/manual/performance-tips/) that could apply, and the [Flux performance tips](https://fluxml.ai/Flux.jl/stable/guide/performance/). In particular, some of the custom structs in this package could be made more efficient by simply adding type parameters, as discusses in the [Flux's custom model example](https://fluxml.ai/Flux.jl/stable/tutorials/custom_layers/#Custom-Model-Example).
3131

3232
### Refactoring
33-
- [ ] Refactor code so that all methods for a given estimator class are in a single file (this will be much clearer, making contributing and long-term maintenance easier)
33+
- [ ] Refactor code so that all methods for a given estimator class are in a single file (this will make contributing and long-term maintenance easier)
34+
- [ ] Remove GraphNeuralNetworks.jl and related packages as dependencies to reduce installation time (ideally, make the GNN functionality an extension).
3435
- [ ] Move [DeepSet](https://msainsburydale.github.io/NeuralEstimators.jl/dev/API/architectures/#NeuralEstimators.DeepSet) to Flux.jl
3536
- [ ] For long term stability, it might be better to use Plots.jl, rather than AlgebraOfGraphics.jl and CairoMakie.jl
3637

0 commit comments

Comments
 (0)