Closed shihchengli closed 1 year ago
@shihchengli Thanks! Looks good. FiltIDs are now removed from the required mol attributes. A double-check was done during the review to make sure all 'FiltIDs' assignments were cleaned. Shouldn't be an issue loading molecule calculation results with 'FiltIDs'
In the PR in #34, a
FilterIDs
is added in RDKitMol object during TS conformers generation, but theKeepIDs
can actually do whatFilterIDs
is supposed to do. Hence, theFilterIDs
is removed in this PR.