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

dandi organize command error #904

Closed
Saksham20 opened this issue Feb 10, 2022 · 4 comments
Closed

dandi organize command error #904

Saksham20 opened this issue Feb 10, 2022 · 4 comments

Comments

@Saksham20
Copy link
Contributor

when running tests for organising video files: this test, this error is produced:

ImportError while loading conftest '/Users/sakshamsharda/Documents/NWB/dandi-cli/dandi/conftest.py'.
dandi/conftest.py:7: in <module>
    from .tests.fixtures import *  # noqa: F401, F403  # lgtm [py/polluting-import]
dandi/tests/fixtures.py:34: in <module>
    from ..cli.command import organize
dandi/cli/command.py:143: in <module>
    from .cmd_download import download  # noqa: E402
dandi/cli/cmd_download.py:7: in <module>
    from ..dandiarchive import _dandi_url_parser, parse_dandi_url
dandi/dandiarchive.py:48: in <module>
    from .dandiapi import BaseRemoteAsset, DandiAPIClient, RemoteDandiset
dandi/dandiapi.py:1524: in <module>
    class RemoteBlobAsset(RemoteAsset, BaseRemoteBlobAsset):
pydantic/main.py:367: in pydantic.main.ModelMetaclass.__new__
    ???
/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/abc.py:85: in __new__
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
E   TypeError: multiple bases have instance lay-out conflict
@jwodder
Copy link
Member

jwodder commented Feb 10, 2022

@Saksham20 What versions of Python and pydantic are you using?

@Saksham20
Copy link
Contributor Author

python 3.8.9
pydantic 1.8.2

@jwodder
Copy link
Member

jwodder commented Feb 10, 2022

It turns out the master branch of dandi requires pydantic 1.9.0 or higher. I've created a PR to update the dependencies (#905).

@Saksham20 Please update your version of pydantic and try again.

@Saksham20
Copy link
Contributor Author

thanks, it works now

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