You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, adding "layers" or non-indexed atoms is done with a recursive method in the fragment_graphs and cluster_graphs. This is also true for making SMIRKS patterns for both new graphs here and ChemicalEnvironments. I tend to think that while loops are better (slightly less dangerous) than recursive methods. It would be nice to try to write these functions as loops instead of recursively.
The text was updated successfully, but these errors were encountered:
I have gotten mixed feedback from people about if while loops are actually preferable to recursive functions, but it is something for the person taking over this project to decide how they feel about it.
Currently, adding "layers" or non-indexed atoms is done with a recursive method in the
fragment_graphs
andcluster_graphs
. This is also true for making SMIRKS patterns for both new graphs here andChemicalEnvironments
. I tend to think that while loops are better (slightly less dangerous) than recursive methods. It would be nice to try to write these functions as loops instead of recursively.The text was updated successfully, but these errors were encountered: