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

MergeTree class has no attribute node #4

Open
pattonw opened this issue Dec 10, 2020 · 2 comments
Open

MergeTree class has no attribute node #4

pattonw opened this issue Dec 10, 2020 · 2 comments

Comments

@pattonw
Copy link
Contributor

pattonw commented Dec 10, 2020

MergeTree defined in "lsd/merge_tree.pyx" seems to inherit from nx.DiGraph and tries to look up nodes via self.node[node_id].

I think all of these calls should be changed to self.nodes[node_id].
networkx also isn't in the requirements file so maybe its just a versioning issue.

Ran into this error when using agglomerate_in_block.

@yajivunev
Copy link
Contributor

can confirm, agglomerate_in_block works with networkx==2.2, does not with 2.5

@sheridana
Copy link
Collaborator

I think it makes most sense to move post-processing scripts to funlib.segment or another repo and keep this repo strictly for lsds. we can pin networkx then.

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