Caitlin Bannan

I think the general answer to this is YES, but I will not implement it now. For example, we at least know that extra layers are never necessary for the...

Depending on the number of clusters, we could do this in the purely brute force way where we try moving things around, maybe just reorder randomly 5 times before adding...

We now have more documentation on this in the preprint. However, its not a minor issue. The answer might be that ordering clusters might change depending how you are identifying...

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...

Switching to just `from collections import ...` doesn't work in python 3.6 which we are still supporting. I recommend revisiting this issue after we add python 3.8 support.

[`dask`](https://github.com/dask/dask/pull/3876/files) addresses this by adding its own `compatibility` file which imports in different ways allowing the same package to support multiple versions of python. This might be something to consider...

Yes, there are documented differences in `R`! In OpenEye `R` and `x` are the same, they mean number of ring bonds (or ring connectivity). However, the Daylight SMARTS and RDKit...

This is why document issues is so important so we don't waste time repeating research into the history of a problem. Perhaps there are other things that should be done...

I'm sorry for my curt response, I didn't note the hybridization note before. We (both hand written and auto generated patterns) didn't use the hybridization, so it wouldn't have come...

Looking back at the top issues I have a couple of things to ask: * RDK should be able to parse normal `rn`, so I don't know why `"[r1:1]"` isn't...