@jim22k,test_intersection is raising, and it looks like code you added. Can you help? Note that I implementedintersection in this PR.

Other than my comment above, this all looks good to me.

Other than my comment above, this all looks good to me.

Thanks for reviewing! Thoughts on the failing test?

Thoughts on the failing test?

I think the line:
if os.environ.get("NETWORKX_GRAPH_CONVERT", None) != "nx-loopback":
should instead be:
if os.environ.get("NETWORKX_GRAPH_CONVERT", None) is not None:

The point of that test is to check thatnx._dispatch raises aTypeError when differently typed Graph objects are passed in. But we don't need to test that during auto dispatching tests.

Can you make that change in NetworkX and see if the tests pass? If so, please add that change toyour NetworkX PR. Thanks!

if os.environ.get("NETWORKX_GRAPH_CONVERT", None) is not None:

This doesn't work, b/c we convert networkx graphs (we always call e.g.to_graph). Should it instead be== "nx-loopback":? I still don't quite follow what's going on. Also, I think this ought to be broken out into its own test.

Oops. Try this instead:

# Only test if not performing auto convert testing of backend implementationsifos.environ.get("NETWORKX_GRAPH_CONVERT",None)isNone:

The goal is to only test this if wearen't doing auto convert testing.

Merging. The test failure is understood--we are waiting on NetworkX PR to be merged.