InformaticsMatters / squonk

Squonk platform and computational notebook
Apache License 2.0
8 stars 3 forks source link

Wrong class names for chemcentral tables were present. #146

Closed tdudgeon closed 3 years ago

tdudgeon commented 3 years ago

The chemcentral search was broken, caused by this commit: https://github.com/InformaticsMatters/squonk/commit/d26900d971d10257869e0daab188fae6c54ce200 That commit removed duplicate classes for tables, but the loader has already loaded the data using the classes that were removed resulting in the data in the table_info having the name of the deleted classes (e.g. org.squonk.rdkit.db.tables.MolportTable) rather than the ones that remained (e.g. org.squonk.rdkit.db.impl.MolportTable).

This was fixed by manually updating the contents of the table_info table and I think that next time any data is loaded the correct class names will be used. However until this is done I can't be 100% sure of this. After updating the table contents the chemcentral pod was restarted and now if starts without errors.