Ross Barnowski

Results 296 comments of Ross Barnowski

I'm going to go ahead and close this as it's been stale for a while and still has the "changesets from multiple PRs" issue. @Lukong123 if you're still interested in...

This one has been stale for a while so I'm going to go ahead and close it. @Lukong123 if you are still interested in contributing this notebook, the place to...

@NikitaSharma1 are you still planning to work on this? If so, I think the place to (re)start would be by updating the motivating example per feedback from the previous review!

I'm going to go ahead and close this one since it's been stale for a while. That's not to say a coloring example wouldn't be a valuable addition. @unna97 if...

This one has been stale for a while so I'll go ahead and close it. Thanks for taking a stab @Drishttii , if you're interested in picking this back up...

@F3eQnxN3RriK IMO it's okay to leave this closed - we don't necessarily want to *encourage* churn on the `ma` package.

Thanks all - I'll go ahead and get this in. Once the categories are separated, if anyone would like to add/adjust the admonitions, please follow-up with an issue/PR!

> What do other reviewers think, is this worth pushing forward? It seems a bit wordy to me. My two cents: I agree with @mattip , I personally find the...

A quick ping on this one - IMO the best course of action is to *not* dispatch the `strategy_` functions. AFAICT, those are callables that are intended only for explicit...