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

Same figerprints generated for ECFP and FCFP #114

Open
1 task done
alkorolyov-selvita opened this issue Nov 27, 2024 · 0 comments
Open
1 task done

Same figerprints generated for ECFP and FCFP #114

alkorolyov-selvita opened this issue Nov 27, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@alkorolyov-selvita
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues and found nothing

Bug description

It seems that in the newer versions (0.10.1) of mofleat it generates same values for ecfp and fcfp. Although it works correctly on version 0.8.9.

How to reproduce the bug

import datamol as dm
import numpy as np
from molfeat.calc import FPCalculator

df = dm.data.freesolv()

fcfp = FPCalculator('fcfp')
ecfp = FPCalculator('ecfp')

with dm.without_rdkit_log():
    X_fcfp = np.stack(df.smiles.apply(fcfp))
    X_ecfp = np.stack(df.smiles.apply(ecfp))

(X_fcfp == X_ecfp).all()

True

Error messages and logs

results are equal for different calculators

Environment

Current environment
- Molfeat version (e.g., 0.1.0): 0.10.1
- RDKit version (e.g., 2022.09.5):  2024.09.2
- OS (e.g., Linux): Ubuntu 22.04
- How you installed Molfeat (`conda`, `pip`, source): conda

Additional context

As mentioned above, it works correctly in molfeat version 0.8.9

@alkorolyov-selvita alkorolyov-selvita added the bug Something isn't working label Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant