Skip to content

Commit 8b2b668

Browse files
committed
Useful debugging output from CoordinationTransformer. Could think about changing TreeGraphNode to print out the whole tree, but that would presumably mess up some various dependency outputs
1 parent 3a33fef commit 8b2b668

File tree

1 file changed

+24
-16
lines changed

1 file changed

+24
-16
lines changed

src/edu/stanford/nlp/trees/CoordinationTransformer.java

+24-16
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,14 @@ public CoordinationTransformer(HeadFinder hf, boolean performMWETransformation)
7171
qp = new QPTreeTransformer(performMWETransformation);
7272
}
7373

74+
public void debugLine(String prefix, Tree t) {
75+
if (t instanceof TreeGraphNode) {
76+
log.info(prefix + ((TreeGraphNode) t).toOneLineString());
77+
} else {
78+
log.info(prefix + t);
79+
}
80+
}
81+
7482
/**
7583
* Transforms t if it contains a coordination in a flat structure (CCtransform)
7684
* and transforms UCP (UCPtransform).
@@ -81,19 +89,19 @@ public CoordinationTransformer(HeadFinder hf, boolean performMWETransformation)
8189
@Override
8290
public Tree transformTree(Tree t) {
8391
if (VERBOSE) {
84-
log.info("Input to CoordinationTransformer: " + t);
92+
debugLine("Input to CoordinationTransformer: ", t);
8593
}
8694

8795
if (performMWETransformation) {
8896
t = gappingTransform(t);
8997
if (VERBOSE) {
90-
log.info("After t = gappingTransform(t);\n: " + t);
98+
debugLine("After t = gappingTransform(t);: ", t);
9199
}
92100
}
93101

94102
t = tn.transformTree(t);
95103
if (VERBOSE) {
96-
log.info("After DependencyTreeTransformer: " + t);
104+
debugLine("After DependencyTreeTransformer: ", t);
97105
}
98106
if (t == null) {
99107
return t;
@@ -102,59 +110,59 @@ public Tree transformTree(Tree t) {
102110
if (performMWETransformation) {
103111
t = MWETransform(t);
104112
if (VERBOSE) {
105-
log.info("After MWETransform: " + t);
113+
debugLine("After MWETransform: ", t);
106114
}
107115

108116
t = MWFlatTransform(t);
109117
if (VERBOSE) {
110-
log.info("After MWFlatTransform: " + t);
118+
debugLine("After MWFlatTransform: ", t);
111119
}
112120

113121
t = prepCCTransform(t);
114122
if (VERBOSE) {
115-
log.info("After prepCCTransform: " + t);
123+
debugLine("After prepCCTransform: ", t);
116124
}
117125
}
118126

119127
t = UCPtransform(t);
120128
if (VERBOSE) {
121-
log.info("After UCPTransformer: " + t);
129+
debugLine("After UCPTransformer: ", t);
122130
}
123131
t = CCtransform(t);
124132
if (VERBOSE) {
125-
log.info("After CCTransformer: " + t);
133+
debugLine("After CCTransformer: ", t);
126134
}
127135
t = qp.transformTree(t);
128136
if (VERBOSE) {
129-
log.info("After QPTreeTransformer: " + t);
137+
debugLine("After QPTreeTransformer: ", t);
130138
}
131139
t = SQflatten(t);
132140
if (VERBOSE) {
133-
log.info("After SQ flattening: " + t);
141+
debugLine("After SQ flattening: ", t);
134142
}
135143
t = dates.transformTree(t);
136144
if (VERBOSE) {
137-
log.info("After DateTreeTransformer: " + t);
145+
debugLine("After DateTreeTransformer: ", t);
138146
}
139147
t = removeXOverX(t);
140148
if (VERBOSE) {
141-
log.info("After removeXoverX: " + t);
149+
debugLine("After removeXoverX: ", t);
142150
}
143151
t = combineConjp(t);
144152
if (VERBOSE) {
145-
log.info("After combineConjp: " + t);
153+
debugLine("After combineConjp: ", t);
146154
}
147155
t = moveRB(t);
148156
if (VERBOSE) {
149-
log.info("After moveRB: " + t);
157+
debugLine("After moveRB: ", t);
150158
}
151159
t = changeSbarToPP(t);
152160
if (VERBOSE) {
153-
log.info("After changeSbarToPP: " + t);
161+
debugLine("After changeSbarToPP: ", t);
154162
}
155163
t = rearrangeNowThat(t);
156164
if (VERBOSE) {
157-
log.info("After rearrangeNowThat: " + t);
165+
debugLine("After rearrangeNowThat: ", t);
158166
}
159167

160168
return t;

0 commit comments

Comments
 (0)