https://github.com/mosdef-hub/gmso/pull/492
- Function `select_params_on_networkx`: Its not clear to be why we need so many if/else ladders. Having said that there should be a way to consolidate these ladders into one by employing the signature like select_params_on_networkx(networkx_graph, *atoms)
It might be not be necessary to cast the networkx nodes to a list if you don’t want to reiterate through it. It already returns an iterator