Ori Press

Results 15 comments of Ori Press

Hi, As the task is currently defined, the algorithm is valid. You're correct in that if that input data was float64, the proposed algorithm wouldn't work. As that is currently...

Hi, thanks for bringing this to our attention. Can you please try pulling from remote and trying again?

Hi, I'm glad its working for you. 7 hours is completely normal. For the paper, we used a Slurm cluster with multiple cpu nodes. That is what we recommend. When...

Thanks for bringing this to my attention. I have just pushed some changes that should fix it. Please do not hesitate to report further issues.

Hi! This PR adds functionality but I don't have permissions to add a label. Could a maintainer please add the type: Enhancements label? Thanks!

Sure thing @amcandio, fixed in the latest commit.

I implemented the requested changes in the latest commit @amcandio. Thanks for the feedback!

Would it be possible to merge this change? If not, what are the fixes needed? Thanks!

I ran some ablations based on your feedback: - `upstream_main`: current NetworkX main. - `upstream_main+early_break`: upstream plus early exit at lines 275,282. - `upstream_main+early_break+custom_dsu`: adds custom DSU at lines 238-262....