This issue is for discussing the removal of query_id from NodeBinding (formal deprecation).
Currently, some members are in favor of removal, recalling...
- in the past,
query_id was used for QNode ID subclassing cases: the NodeBinding's query_id would be the original QNode ID, while the the id would be descendant Node that's actually used in the Result.
- that behavior has been replaced by using AuxGraphs: the NodeBinding
id is to the QNode ID, and the edge has a support_graph (AuxGraph) with the subclassing edges/descendant Node that's actually used
- so the
query_id isn't used for anything
Other members want more discussion, recalling...
query_id is used for set-input queries?
This issue is for discussing the removal of
query_idfrom NodeBinding (formal deprecation).Currently, some members are in favor of removal, recalling...
query_idwas used for QNode ID subclassing cases: the NodeBinding'squery_idwould be the original QNode ID, while the theidwould be descendant Node that's actually used in the Result.idis to the QNode ID, and the edge has a support_graph (AuxGraph) with the subclassing edges/descendant Node that's actually usedquery_idisn't used for anythingOther members want more discussion, recalling...
query_idis used for set-input queries?