-
Couldn't load subscription status.
- Fork 6
Open
Labels
featureNew feature or extensionNew feature or extension
Description
... which is bad for autodiff since reverse passes are more expensive than forward passes.
I wonder if this could be due to the order in which we iterate through the stars / trees? It doesn't seem to be influenced by the order in which we greedily color vertices.
A = diagm(ones(Bool, 3))
problem = ColoringProblem{:nonsymmetric,:bidirectional}()
algo = GreedyColoringAlgorithm(RandomOrder(); postprocessing=true)
for _ in 1:10
result = coloring(A, problem, algo)
@show column_colors(result), row_colors(result)
endPerhaps in this line
SparseMatrixColorings.jl/src/coloring.jl
Line 646 in 7efc6bc
| for j in axes(S, 2) |
we could use the computed vertex order instead, or its reverse order? Not sure which one makes more sense?
I noticed this because in JuliaDiff/DifferentiationInterface.jl#864 I sometimes got empty forward groups but never empty reverse groups.
Metadata
Metadata
Assignees
Labels
featureNew feature or extensionNew feature or extension