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 #85

Open
jbloom opened this issue Mar 27, 2023 · 4 comments · Fixed by #87
Open

deprecation warning #85

jbloom opened this issue Mar 27, 2023 · 4 comments · Fixed by #87

Comments

@jbloom
Copy link
Member

jbloom commented Mar 27, 2023

The code currently gives this warning:

/fh/fast/bloom_j/software/miniconda3/envs/dms-vep-pipeline/lib/python3.11/site-packages/dms_variants/codonvarianttable.py:1779: FutureWarning: In a future version, the Index constructor will not infer numeric dtypes when passed object-dtype sequences (matching Series behavior)
@jbloom jbloom linked a pull request Feb 6, 2024 that will close this issue
@jbloom jbloom closed this as completed in #87 Feb 6, 2024
Copy link
Member Author

jbloom commented Feb 6, 2024

Not yet addressed, closed by accident.

@jbloom jbloom reopened this Feb 6, 2024
@jgallowa07
Copy link

I think this might be related but I just installed dms_variants == 1.6.0 and when I tried

import dms_variants.codon_variant_table

I got

    from mizani.utils import multitype_sort
ImportError: cannot import name 'multitype_sort' from 'mizani.utils' (/home/jgallowa/miniforge3/envs/multidms-dev/lib/python3.9/site-packages/mizani/utils.py)

The underlying problem seems to be with plotnine. Looking into a solution for this now.

@jgallowa07
Copy link

It works for me if I pin mizani, a dependency of plotnine, to 011.4. Maybe worth raising an issue with plotnine, as the problem is coming from there.

@jbloom
Copy link
Member Author

jbloom commented Sep 19, 2024

Yes, do you want to raise the plotnine issue?

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

Successfully merging a pull request may close this issue.

2 participants