I believe I have a fix for this now. Just to make sure we're all on the same page about it, with this fix
If all of the support graphs on an edge get removed for blocklist reasons, we remove that edge
If that edge is part of any analyses, it is removed from there as well
If removing edges from an analysis leaves us with an analysis that has no edges, we remove that analysis
If removing that analysis leaves us with a result which has no analyses, we remove that result.
All said, just to give an idea of numbers it looks like these are the differences in the number of things removed from Unsecret's merged response pre vs post fix:
Nodes: 25 -> 25
Edges: 289 -> 386
Results: 24 -> 105
Aux graphs: 128 -> 128
Analyses: 16->99
Lot going on in this one, but pasting my comment from the relevant ticket in Feedback https://github.com/NCATSTranslator/Feedback/issues/784
I believe I have a fix for this now. Just to make sure we're all on the same page about it, with this fix
All said, just to give an idea of numbers it looks like these are the differences in the number of things removed from Unsecret's merged response pre vs post fix: Nodes: 25 -> 25 Edges: 289 -> 386 Results: 24 -> 105 Aux graphs: 128 -> 128 Analyses: 16->99