Isn’t find_edges part of the normal Uniform Negative Sampler implementation?
My bad. You are right. Also for your implementation, the negative edges do not necessary need to have the same source nodes as the positive edges.
By this, are you now saying that we should choose negative edges and aim to minimize their score and have positive edges as the new “negative examples”? Basically doing the opposite of the link prediction negative sampler? Would this have a benefit over the previous method?
No. There will be two EdgeDataLoader instances sampling positive edges only. It’s just that the first EdgeDataLoader
treats real edges as positive edges and the second EdgeDataLoader
treats the specific edges as positive edges. I’m not sure which approach will be more clean or efficient.