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

Bug with GMatch4py if start with an other example #24

Open
pkrezel opened this issue Nov 20, 2020 · 4 comments
Open

Bug with GMatch4py if start with an other example #24

pkrezel opened this issue Nov 20, 2020 · 4 comments

Comments

@pkrezel
Copy link

pkrezel commented Nov 20, 2020

Hi,
I noticed that if I don't begin with the examples given in your document, GMatch4py fails and gives:
_


TypeError Traceback (most recent call last)
in
----> 1 result=ged.compare([Gx1, Gx2, Gx3],None)
2 print(result)

gmatch4py/ged/abstract_graph_edit_dist.pyx in gmatch4py.ged.abstract_graph_edit_dist.AbstractGraphEditDistance.compare()

gmatch4py/ged/abstract_graph_edit_dist.pyx in gmatch4py.ged.abstract_graph_edit_dist.AbstractGraphEditDistance.compare()

gmatch4py/helpers/general.pyx in gmatch4py.helpers.general.parsenx2graph()

gmatch4py/graph.pyx in gmatch4py.graph.Graph.init()

TypeError: Expected str, got int

_

If I begin with your example and do the same thing it works!

@jacquesfize
Copy link
Owner

jacquesfize commented Nov 20, 2020

Hi @pkrezel,

Can you show me a sample of your graphs ?

Best,
@Jacobe2169

@pkrezel
Copy link
Author

pkrezel commented Nov 20, 2020

Hi,
I attached a pickle containing the list of networkx graphs.
can you read it ?
[
LGx.zip

](url)

@pkrezel
Copy link
Author

pkrezel commented Nov 20, 2020

Hi,
it seems that the problem is linked with the definition of the attributes to be used:
ged.set_attr_graph_used("color", "weight")

I attached the new pickle with graphs containing "weight" as attribute.
LGx.zip

@pkrezel
Copy link
Author

pkrezel commented Nov 20, 2020

Hi,
I think it is a mistake coming from me.
In my case, I don't have to worry with "ged.set_attr_graph_used" because I want to use all the elements of the graph.

Best.

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