Skip to content

Commit 7e9e95f

Browse files
committed
Update documentation now that the position is correctly handled in AddDep
1 parent 2cdd6e5 commit 7e9e95f

File tree

1 file changed

+5
-5
lines changed
  • src/edu/stanford/nlp/semgraph/semgrex/ssurgeon

1 file changed

+5
-5
lines changed

src/edu/stanford/nlp/semgraph/semgrex/ssurgeon/AddDep.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,11 @@
1616

1717
/**
1818
* Adds a new dependent node, based off of a prototype IndexedWord, with the given relation.
19-
* The new node's sentence index is inherited from the governing node. Currently a cheap heuristic
20-
* is made, placing the new node as the leftmost child of the governing node.
19+
* The new node's sentence index is inherited from the governing node.
20+
*<br>
21+
* Nodes will be placed at the end of the sentence by default, at the
22+
* start or end using "-position -" or "-position +", or
23+
* before or after a node with "-position -word" and "-position +word"
2124
*
2225
* @author Eric Yeh
2326
*
@@ -137,9 +140,6 @@ public static void moveNodes(SemanticGraph sg, Function<Integer, Boolean> should
137140
}
138141

139142
/**
140-
* TODO: figure out how to specify where in the sentence this node goes.
141-
* currently allows - and + for start and end. before and after
142-
* matched node would be good
143143
* TODO: bombproof if this gov, dep, and reln already exist.
144144
*/
145145
@Override

0 commit comments

Comments
 (0)