From 4b2e418ddce89add1cd9b5259074d5fd87c32692 Mon Sep 17 00:00:00 2001 From: Starbeamrainbowlabs Date: Fri, 11 Nov 2022 18:55:35 +0000 Subject: [PATCH] finish comment ...oops --- rainfallwrangler/src/lib/manip/array2d_pool.mjs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rainfallwrangler/src/lib/manip/array2d_pool.mjs b/rainfallwrangler/src/lib/manip/array2d_pool.mjs index 4b5f094..91596a6 100644 --- a/rainfallwrangler/src/lib/manip/array2d_pool.mjs +++ b/rainfallwrangler/src/lib/manip/array2d_pool.mjs @@ -7,7 +7,7 @@ export default async function array2d_pool(channels, operator="max") { if(operator !== "max") throw new Error(`Error: Unknown operator '${operator}. At present only the 'max' operator is supported.`); - // This is rather a hack to save time. Tensorflow.js is not needed here, but may result in increased speed. It may be worth rolling this out to the rest of the codebase, thinking about it. While Tensorflow.js hasmany bugs, this only extends to the machine learning / loss functions / models etc and not the + // This is rather a hack to save time. Tensorflow.js is not needed here, but may result in increased speed. It may be worth rolling this out to the rest of the codebase, thinking about it. While Tensorflow.js has many bugs, this only extends to the machine learning / loss functions / models etc and not the fundamental operations. const result_tensor = tf.tidy(() => { const tensor = tf.tensor(channels); return tf.max(tensor, 0, false);