Skip to content

bug(web): epic/boundary-correction does not properly model transpositions 馃敧 #16452

Description

@jahorton

See: #16450

As epic/boundary-correction will also be removing the LegacyQuotientSpur class, which is where the current transposition-handling code resides, we'll need to adapt #16450's contents to better model how the search quotient graph is constructed, placing them where appropriate within its specialized node types.

Suspected development path needed:

this.incomingTransposeRootNodes = [...grandparentNode.previousResults];

As any individual SearchQuotientNode may have multiple parent nodes and/or grandparent nodes in epic/boundary-correction, we'll need to adjust this. The core idea is the same, though - we apply the local inputs after the grandparent path state, then append the parent inputs.

So, we'd want to capture all grandparent sources and request node-path forwarding from them. We'd also want a way to remember which parent quotient nodes follow from which grandparent quotient nodes - likely via some sort of lookup table or hash map. And, of course, a way to remember which grandparent nodes each SearchNode passed through. (So, SearchNode -> grandparent node it came from -> allowed parent quotient node(s) -> the input(s) to transpose.) .spaceID values may prove useful here for linking everything together.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    • Status
      Todo

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions