Executes a single addEdge. Perhaps need to investigate making this work multiple times over a single graph instead of returning all possible single modifications of the graph

Author: AngledLuffa

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

@@ -40,6 +40,29 @@
  * loaded can reference these resources.
  *<br>
  * An Ssurgeon can be built from an XML pattern or by assembling the pieces by hand.
+ *<br>
+ * The XML format is as follows:
+<pre>
+{@code
+<ssurgeon-pattern-list>
+  <ssurgeon-pattern>
+    <uid>...</uid>
+    <notes>...</notes>
+    <semgrex>...</semgrex>
+    <edit-list>...</edit-list>
+  </ssurgeon-pattern>
+</ssurgeon-pattern-list>
+}
+</pre>
+ * The {@code id} is the id of the Ssurgeon operation. <br>
+ * The {@code notes} are comments on the Ssurgeon. <br>
+ * The {@code semgrex} is a Semgrex pattern to use when matching for this operation. <br>
+ * The {@code edit-list} is the actual Ssurgeon operation to execute. <br>
+ *
+ * Available operations and their arguments include:
+ * <ul>
+ * <li> {@code addEdge -gov a1 -dep a2 -reln dep -weight 0.5}
+ * </ul>
  *
  * @author Eric Yeh
  */
@@ -242,7 +265,7 @@ protected static class SsurgeonArgs {
     // below are string representations of the intended values
     public String nodeString = null;
 
-    public double weight = 1.0;
+    public double weight = 0.0;
 
     public String name = null;
   }
@@ -333,7 +356,7 @@ public static SsurgeonEdit parseEditLine(String editLine) {
     } else if (command.equalsIgnoreCase(AddNode.LABEL)) {
       retEdit = AddNode.createAddNode(argsBox.nodeString, argsBox.name);
     } else if (command.equalsIgnoreCase(AddEdge.LABEL)) {
-      retEdit = AddEdge.createEngAddEdge(argsBox.govNodeName, argsBox.dep, argsBox.reln);
+      retEdit = AddEdge.createEngAddEdge(argsBox.govNodeName, argsBox.dep, argsBox.reln, argsBox.weight);
     } else if (command.equalsIgnoreCase(DeleteGraphFromNode.LABEL)) {
       retEdit = new DeleteGraphFromNode(argsBox.node);
     } else if (command.equalsIgnoreCase(RemoveEdge.LABEL)) {

test/src/edu/stanford/nlp/semgraph/semgrex/ssurgeon/SsurgeonTest.java

@@ -28,6 +28,31 @@ public void readXMLEmptyPattern() {
                              "</ssurgeon-pattern-list>");
     Ssurgeon inst = Ssurgeon.inst();
     List<SsurgeonPattern> pattern = inst.readFromString(doc);
+    assertEquals(pattern.size(), 1);
+  }
+
+  @Test
+  public void readXMLEdit() {
+    String newline = System.getProperty("line.separator");
+    String doc = String.join(newline,
+                             "<ssurgeon-pattern-list>",
+                             "  <ssurgeon-pattern>",
+                             "    <uid>38</uid>",
+                             "    <notes>This is a simple test of addEdge</notes>",
+                             "    <semgrex>{}=a1 &gt; {}=a2</semgrex>",
+                             "    <edit-list>addEdge -gov a1 -dep a2 -reln dep -weight 0.5</edit-list>",
+                             "  </ssurgeon-pattern>",
+                             "</ssurgeon-pattern-list>");
+    Ssurgeon inst = Ssurgeon.inst();
+    List<SsurgeonPattern> patterns = inst.readFromString(doc);
+    assertEquals(patterns.size(), 1);
+    SsurgeonPattern pattern = patterns.get(0);
+
+    SemanticGraph sg = SemanticGraph.valueOf("[A obj> B obj> C]");
+    Collection<SemanticGraph> newSgs = pattern.execute(sg);
+    // TODO: perhaps it would be better to have an execution scheme
+    // where one graph has all possible modifications applied
+    assertEquals(newSgs.size(), 2);
   }
 
   /**