Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RDKit Deprecation Warnings for MorganFingerprint implementations #41

Closed
RiesBen opened this issue Aug 1, 2024 · 4 comments
Closed

RDKit Deprecation Warnings for MorganFingerprint implementations #41

RiesBen opened this issue Aug 1, 2024 · 4 comments

Comments

@RiesBen
Copy link
Contributor

RiesBen commented Aug 1, 2024

Hi,
not sure if that was already observed somewhere else, but I just recently stumbled over these warning:

[10:22:21] DEPRECATION WARNING: please use MorganGenerator
[10:22:21] DEPRECATION WARNING: please use MorganGenerator
[10:22:21] DEPRECATION WARNING: please use MorganGenerator
[10:22:21] DEPRECATION WARNING: please use MorganGenerator
[10:22:21] DEPRECATION WARNING: please use MorganGenerator
[10:22:21] DEPRECATION WARNING: please use MorganGenerator
[10:22:21] DEPRECATION WARNING: please use MorganGenerator
[10:22:21] DEPRECATION WARNING: please use MorganGenerator

I assume there is an RDKit change upcoming?

was using from conda-forge:

  Name      Version    Build            Channel    
─────────────────────────────────────────────────────
  librdkit  2024.03.5  h79cfef2_2       conda-forge
  rdkit     2024.03.5  py312h7b4b7d0_2  conda-forge

and via PyPi:

scikit_mol                    0.3.1
@RiesBen RiesBen changed the title RDKit Deapprecations RDKit Deprecations for MorganFingerprint implementations Aug 1, 2024
@RiesBen RiesBen changed the title RDKit Deprecations for MorganFingerprint implementations RDKit Deprecation Warnings for MorganFingerprint implementations Aug 1, 2024
@EBjerrum
Copy link
Owner

EBjerrum commented Aug 8, 2024

Thanks for the heads-up. didn't see this warning.

@EBjerrum
Copy link
Owner

EBjerrum commented Aug 9, 2024

There'll be a redesigned base class needed and refactoring of most fingerprints . On the other hand, it will allow a more consistent implementation and opens up for some new features. A good overview of the options are in Gregs blogpost here:
https://greglandrum.github.io/rdkit-blog/posts/2023-01-18-fingerprint-generator-tutorial.html

@RiesBen
Copy link
Contributor Author

RiesBen commented Sep 13, 2024

Hackathon TIME ! :)

@EBjerrum
Copy link
Owner

Solved with #54 . Thanks for the effort!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants