Ross Barnowski

Results 471 comments of Ross Barnowski

Thanks for bringing this up @msschwartz21 ! I'm not sure exactly what's going on... the [deployed docs](https://deepcell.readthedocs.io/en/master/API/deepcell.html) (which are built from `master` rather than latest stable) appear to have all...

Very strange - it looks like [the docs preview from #721](https://deepcell--721.org.readthedocs.build/en/721/API/deepcell.html) is now rendered correctly... maybe RTD had a temporary hiccup or something weird happened where an incomplete build was...

I went ahead and split up the benchmark in a naive way in order to unstick this - maybe @MridulS can doublecheck that this makes sense. Anyways, my approval of...

The CI failures are unrelated, so I'll go ahead and get this in. Thanks again @mjschwenne !

Thanks for updating the benchmarking results @amcandio ! They give me pause though: the speedups for the long paths are obvious and undisputed, but it also looks like there are...

This feature request was taken up in #14989 - there's a lot of good discussion there on this topic. I just wanted to chime in here that you can already...

@mrecachinas thanks for this PR and #3886! There will be a NetworkX release in the near future (O[weeks]) which will have `panther_similarity` in it. One thing we would like to...

Re: naming, I agree with @dschult and @storopoli : **Panther** -> `panther_similarity` and **Panther++** -> `panther_similarity_pp`. > I think we should leave generate_random_paths where it is until NXEP3 is figured...

TL;DR - I agree with @dschult , I'm +1 on having `generate_random_paths` remain fully public (no `_`-prepended) for 2.6 and beyond. I think the points in @dschult 's points are...

I will close this as hopefully *improved* by the examples added in #7627 - any other ideas for how the documentation can be improved are of course welcome!