Merge remote-tracking branch 'refs/remotes/origin/fix/gnnlux-mooncake-skip' into fix/gnnlux-mooncake-skip

Author: Parvm1102

.github/workflows/test_GNNGraphs.yml

@@ -44,7 +44,7 @@ jobs:
       - uses: julia-actions/julia-processcoverage@v1
         with:
           directories: GNNGraphs/src,GNNGraphs/ext
-      - uses: codecov/codecov-action@v5
+      - uses: codecov/codecov-action@v6
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           files: lcov.info

.github/workflows/test_GNNLux.yml

@@ -42,7 +42,7 @@ jobs:
       - uses: julia-actions/julia-processcoverage@v1
         with:
           directories: GNNLux/src,GNNGraphs/src,GNNGraphs/ext,GNNlib/src,GNNlib/ext
-      - uses: codecov/codecov-action@v5
+      - uses: codecov/codecov-action@v6
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           files: lcov.info

.github/workflows/test_GNNlib.yml

@@ -44,7 +44,7 @@ jobs:
       - uses: julia-actions/julia-processcoverage@v1
         with:
           directories: GNNlib/src,GNNlib/ext,GNNGraphs/src,GNNGraphs/ext
-      - uses: codecov/codecov-action@v5
+      - uses: codecov/codecov-action@v6
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           files: lcov.info

.github/workflows/test_GraphNeuralNetworks.yml

@@ -42,7 +42,7 @@ jobs:
       - uses: julia-actions/julia-processcoverage@v1
         with:
           directories: GraphNeuralNetworks/src,GNNGraphs/src,GNNGraphs/ext,GNNlib/src,GNNlib/ext
-      - uses: codecov/codecov-action@v5
+      - uses: codecov/codecov-action@v6
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
           files: lcov.info

GNNGraphs/src/transform.jl

@@ -65,7 +65,7 @@ end
 
 function remove_self_loops(g::GNNGraph{<:ADJMAT_T})
     @assert isempty(g.edata)
-    A = g.graph
+    A = copy(g.graph)
     A[diagind(A)] .= 0
     if A isa AbstractSparseMatrix
         dropzeros!(A)

GNNGraphs/test/transform.jl

@@ -376,6 +376,20 @@ end
             @test size(get_edge_weight(g2)) == (g2.num_edges,)
             @test size(g2.edata.e1) == (3, g2.num_edges)
             @test size(g2.edata.e2) == (g2.num_edges,)
+        else
+            A = [1 1 0
+                 0 1 1
+                 1 0 0]
+            A_no_loops = [0 1 0
+                          0 0 1
+                          1 0 0]
+            g = GNNGraph(A; graph_type = GRAPH_T)
+            g2 = remove_self_loops(g)
+
+            @test Matrix(adjacency_matrix(g)) == A
+            @test Matrix(adjacency_matrix(g2)) == A_no_loops
+            @test g2.num_edges == count(!iszero, A_no_loops)
+            @test g.graph !== g2.graph
         end 
     end
 end