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

Matcher dictionaries for lookup #169

Open
msschwartz21 opened this issue Nov 25, 2024 · 1 comment
Open

Matcher dictionaries for lookup #169

msschwartz21 opened this issue Nov 25, 2024 · 1 comment
Labels
enhancement New feature or request topic: matcher

Comments

@msschwartz21
Copy link
Collaborator

  • Issue 1: We are constantly re-creating dictionaries from gt to pred and pred to gt nodes (and edges).
    • Proposal: Store those dictionaries in the matched object. Or… on the nodes and edges? What is the best way to represent a matching between two graphs? I know we were wary of storing redundant information, but I think the efficiency trade-off is probably worth it, since we often compute the dicts anyways.
  • Issue 2: I need edge matching to implement my matcher. It is not fully implied by/equivalent to a node matching.
    • Proposal: Store an edge matching along with the node matching
  • Dictionary from gt → pred and pred → gt nodes and edges in matched object
@DragaDoncila
Copy link
Collaborator

DragaDoncila commented Nov 25, 2024

Would we consider just storing the matched node/edge on the graph itself after matching? I guess the issue there arises where you do multiple matching, since we don't want to copy the graph, just store a reference. Still the matcher could store an attribute which is the match_key that tells you the name of the node/edge attribute to query. I ask because otherwise to dump all the metric info to file, one has to dump the graphs and the dictionaries, whereas if stored on the graph, you could just dump the graph. And the match key I guess... Lol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request topic: matcher
Projects
Status: No status
Development

No branches or pull requests

2 participants