Add some comments on MergeNodes

AngledLuffa · AngledLuffa · commit 50fdd771d66a · 2025-05-16T23:27:27.000-07:00
diff --git a/src/edu/stanford/nlp/semgraph/semgrex/ssurgeon/MergeNodes.java b/src/edu/stanford/nlp/semgraph/semgrex/ssurgeon/MergeNodes.java
@@ -142,6 +142,9 @@ public boolean evaluate(SemanticGraph sg, SemgrexMatcher sm) {
       newLabel.setBefore(right.before());
     }
 
+    // find the head, and replace all the existing annotations on the head
+    // with the new annotations (including word and lemma)
+    // from the newly built CoreLabel
     for (IndexedWord vertex : sg.vertexSet()) {
       if (vertex.index() == head.index()) {
         for (Class key : newLabel.keySet()) {
@@ -151,6 +154,7 @@ public boolean evaluate(SemanticGraph sg, SemgrexMatcher sm) {
       }
     }
 
+    // delete the dependency
     // copy the list so that deletion doesn't hurt the iterator
     // TODO: super fancy would be implementing iterator.remove()
     // on the Set returned by the SemanticGraph

Original file line number	Diff line number	Diff line change
`@@ -142,6 +142,9 @@ public boolean evaluate(SemanticGraph sg, SemgrexMatcher sm) {`
`142`	`142`	`newLabel.setBefore(right.before());`
`143`	`143`	`}`
`144`	`144`
	`145`	`+ // find the head, and replace all the existing annotations on the head`
	`146`	`+ // with the new annotations (including word and lemma)`
	`147`	`+ // from the newly built CoreLabel`
`145`	`148`	`for (IndexedWord vertex : sg.vertexSet()) {`
`146`	`149`	`if (vertex.index() == head.index()) {`
`147`	`150`	`for (Class key : newLabel.keySet()) {`
`@@ -151,6 +154,7 @@ public boolean evaluate(SemanticGraph sg, SemgrexMatcher sm) {`
`151`	`154`	`}`
`152`	`155`	`}`
`153`	`156`
	`157`	`+ // delete the dependency`
`154`	`158`	`// copy the list so that deletion doesn't hurt the iterator`
`155`	`159`	`// TODO: super fancy would be implementing iterator.remove()`
`156`	`160`	`// on the Set returned by the SemanticGraph`