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

Update pyvo version to 1.5.1 #1315

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Conversation

stvoutsin
Copy link
Collaborator

Pyvo in the current main branch is broken
Issue:
#1314

With this upgrade the import works again:

Deploy new GaiaDMP with version on main

ssh into Zeppelin node:

Try to import pyvo in python3

python3

import pyvo

    Traceback (most recent call last):
    File "", line 1, in
    File "/usr/local/lib/python3.11/site-packages/pyvo/init.py", line 37, in
    from . import registry
    File "/usr/local/lib/python3.11/site-packages/pyvo/registry/init.py", line 7, in
    from . import regtap
    File "/usr/local/lib/python3.11/site-packages/pyvo/registry/regtap.py", line 19, in
    from ..dal import scs, sia, ssa, sla, tap, query as dalq
    File "/usr/local/lib/python3.11/site-packages/pyvo/dal/init.py", line 2, in
    from .sia import search as imagesearch
    File "/usr/local/lib/python3.11/site-packages/pyvo/dal/sia.py", line 39, in
    from .query import DALResults, DALQuery, DALService, Record
    File "/usr/local/lib/python3.11/site-packages/pyvo/dal/query.py", line 47, in
    from ..utils.decorators import stream_decode_content
    File "/usr/local/lib/python3.11/site-packages/pyvo/utils/decorators.py", line 4, in
    from astropy.utils.decorators import wraps
    ImportError: cannot import name 'wraps' from 'astropy.utils.decorators'

Upgrade pyvo and try again

pip install pyvo==1.5.1

python3

import pyvo

Success

Copy link
Collaborator

@millingw millingw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine

@millingw millingw merged commit c2f1d7d into wfau:master Mar 1, 2024
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 this pull request may close these issues.

2 participants