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

Python 3.10 incomptablity due to collections #109

Closed
RamonGal opened this issue Feb 8, 2023 · 3 comments
Closed

Python 3.10 incomptablity due to collections #109

RamonGal opened this issue Feb 8, 2023 · 3 comments

Comments

@RamonGal
Copy link

RamonGal commented Feb 8, 2023

After installing the blastreadius package with pip:

pip install blastradius 

It succeeds in installing

But when run on the terraform folder with:

blast-radius --serve .

We get the following trace back, given that I have installed the python version 3.10, where the 'collections' has no attribute 'MutableSet':

Traceback (most recent call last):
  File "/root/gitlab/happyderm/infra-happyDerm/env/bin/blast-radius", line 14, in <module>
    from blastradius.handlers.dot import DotGraph, Format, DotNode
  File "/root/gitlab/happyderm/infra-happyDerm/env/lib/python3.10/site-packages/blastradius/handlers/dot.py", line 12, in <module>
    from blastradius.graph import Graph, Node, Edge
  File "/root/gitlab/happyderm/infra-happyDerm/env/lib/python3.10/site-packages/blastradius/graph.py", line 11, in <module>
    from blastradius.util import Counter
  File "/root/gitlab/happyderm/infra-happyDerm/env/lib/python3.10/site-packages/blastradius/util.py", line 65, in <module>
    class OrderedSet(collections.MutableSet):
AttributeError: module 'collections' has no attribute 'MutableSet'
@Ianyliu
Copy link

Ianyliu commented Feb 8, 2023

@RamonGal
You can try using my fork, which has some minor improvements and features as described here: #103

My fork location: https://github.com/Ianyliu/blast-radius-fork

Test it with the following command:
python3 -m pip install git+https://github.com/Ianyliu/blast-radius-fork

@RamonGal
Copy link
Author

Thanks, I made a dockerfile that ran balstradius with python 3.9

@Ianyliu
Copy link

Ianyliu commented Feb 14, 2023

@RamonGal glad to know it helped!

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

2 participants