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

Deprecation Warning #100

Open
Pantelispanka opened this issue Dec 25, 2021 · 2 comments
Open

Deprecation Warning #100

Pantelispanka opened this issue Dec 25, 2021 · 2 comments

Comments

@Pantelispanka
Copy link

description

Getting some Deprecation warnings from numpy.
DeprecationWarning: Calling np.sum(generator) is deprecated, and in the future will give a different result. Use np.sum(np.fromiter(generator)) or the python sum builtin instead.

site-packages/mordred/ABCIndex.py:88:

minimal reproduction code

Base usage of calc function

    from rdkit import Chem

    from mordred import Chi, ABCIndex

    benzene = Chem.MolFromSmiles('c1ccccc1')

    # create descriptor instance
    abci = ABCIndex.ABCIndex()

    # calculate descriptor value
    result = abci(benzene)

    print(str(abci), result)

    # create descriptor instance with parameter
    chi_pc4 = Chi.Chi(type='path_cluster', order=4)

    # calculate
    result = chi_pc4(benzene)

    print(str(chi_pc4), result)

environment

OS/distribution

Mac os python 3.8

conda or pip

pip

python version

Python 3.8.6

library version

mordred==1.2.0
numpy==1.19.4

@martinwk
Copy link

martinwk commented Dec 1, 2022

This issue is one year old, any chance this will be addressed?

@JacksonBurns
Copy link

This is resolved in our community maintained fork of mordred: https://github.com/JacksonBurns/mordred-community

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

3 participants