josejimenezluna / delfta

Δ-QML for medicinal chemistry
GNU Affero General Public License v3.0
96 stars 16 forks source link

remove explicit offset_idx arguments from calculator methods #70

Closed josejimenezluna closed 3 years ago

josejimenezluna commented 3 years ago

@cisert have a look whenever you can, as this is something you had originally introduced.

cisert commented 3 years ago

Looks good to me. Wasn't too concerned with this being a little ugly since it's mainly in private methods anyways, but it makes more sense like this :)