You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
An example of this problem is running AC on strider_relay_mouse.json.txt (in the repo).
In graph coalesence, we merge N nodes and link that merged set to a common new node.
Sometimes the same N nodes link to multiple different new nodes. We return each of these new nodes as a new coalesced answer.
So if old1, old2 are both linked to new1, new2, we return 2 new answers, one with new1, one with new 2.
That was done primarily so that the rewritten query is simple (we add in one new query node). But it makes looking through the results suboptimal. Really, it would be better to combine n1 and n2 into a single answer. But now there are 2 new nodes in an answer.
Do we need to put 2 qnodes into the query? Can answers that only have one new node just not include a mapping to extra_qnode_2? Or can we leave just a single new qnode and have 2 mappings from one answer to that qnode?
@patrickkwang this gets into a TRAPI issue that I'm not sure the best way forward on.
The text was updated successfully, but these errors were encountered:
None of these options seems unreasonable. I think we just need to make sure the behavior is aligned with the user's expectations for how this operation works.
An example of this problem is running AC on strider_relay_mouse.json.txt (in the repo).
In graph coalesence, we merge N nodes and link that merged set to a common new node.
Sometimes the same N nodes link to multiple different new nodes. We return each of these new nodes as a new coalesced answer.
So if old1, old2 are both linked to new1, new2, we return 2 new answers, one with new1, one with new 2.
That was done primarily so that the rewritten query is simple (we add in one new query node). But it makes looking through the results suboptimal. Really, it would be better to combine n1 and n2 into a single answer. But now there are 2 new nodes in an answer.
Do we need to put 2 qnodes into the query? Can answers that only have one new node just not include a mapping to extra_qnode_2? Or can we leave just a single new qnode and have 2 mappings from one answer to that qnode?
@patrickkwang this gets into a TRAPI issue that I'm not sure the best way forward on.
The text was updated successfully, but these errors were encountered: