Skip to content

Commit 46a362e

Browse files
Deleted TODOs
1 parent 5fc85ac commit 46a362e

File tree

3 files changed

+0
-6
lines changed

3 files changed

+0
-6
lines changed

data.py

-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ def clean_text(self):
2020
"""
2121
Tokenization & string cleaning.
2222
"""
23-
# TODO: NLP preprocessing (use nltk?). Stemming, lemmatization etc.
2423
for i, string in enumerate(self.documents):
2524
string = re.sub(r"[^A-Za-z0-9(),!?'$]", " ", string)
2625
string = re.sub(r"(\d+)", " NUM ", string)

graph_cnn.py

-1
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,6 @@ def graph_max_pool(self, x, p):
217217
return x
218218

219219
def filter_in_fourier(self, x, L, K, F_out, U, W):
220-
# TODO: B x F x V would avoid the permutations?
221220
B, V, F_in = x.get_shape()
222221
B, V, F_in = int(B), int(V), int(F_in)
223222
x = tf.transpose(x, perm=[1, 2, 0]) # V x F_in x B

graph_cnn_train.py

-4
Original file line numberDiff line numberDiff line change
@@ -155,10 +155,6 @@
155155
l2_reg_lambda=l2_reg_lambda)
156156

157157
# Convert sparse matrices to arrays
158-
# TODO: Is there a workaround for this? Doesn't seem memory efficient.
159-
# TODO: https://github.com/tensorflow/tensorflow/issues/342#issuecomment-160354041
160-
# TODO: https://github.com/tensorflow/tensorflow/issues/342#issuecomment-273463729
161-
# TODO: https://stackoverflow.com/questions/37001686/using-sparsetensor-as-a-trainable-variable
162158
x_train = np.squeeze([x_i.toarray() for x_i in x_train])
163159
x_test = np.squeeze([x_i.toarray() for x_i in x_test])
164160

0 commit comments

Comments
 (0)