You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
scikit-learn is only used for PCA to align molecular axes in SmilesWidget. Pulling such a heavy dependency just for this is an overkill. I am not sure if it is even still needed, and I am sure there must be less heavy-weight to do it.
danielhollas
changed the title
Remove scikit-learn, only used for PCA to aligh molecular axis. I am not sure if it is needed, and I am sure there must be less heavy-weight to do it.
Remove scikit-learn dependency
Apr 29, 2024
scikit-learn is only used for PCA to align molecular axes in SmilesWidget. Pulling such a heavy dependency just for this is an overkill. I am not sure if it is even still needed, and I am sure there must be less heavy-weight to do it.
https://stats.stackexchange.com/questions/235882/pca-in-numpy-and-sklearn-produces-different-results
Past PRs for context:
#160
#164
The text was updated successfully, but these errors were encountered: