-
Notifications
You must be signed in to change notification settings - Fork 2
Description
If we want to make this a broad NMR kit, we might get into issues with dependencies and their versions. Right now, I try to add a prediction model which was done previously. If I try to run it with a set of current software versions, I get an error, which seems to be due to a change in RDkit:
conformer = mol.GetConformer()[conf_i]
needs to become
conformer = mol.GetConformer(conf_i.item())
conf_i is a numpy type, the item() converts to int.
This workedin rdkit 2020.09.1.0
If I fix this, I get another error - the error comes from the code I want to import, and it is a wrong number of elements in a list. Whilst this is not directly due to versions, it does work in a different combination of library versions.
I think we will get into this issue more frequently - if we want to use existing code. Ideally, we would isolate the components. I am not an expert on docker, but can't docker do that (if we set it up)?