Mercurial > repos > public > sbplib_julia

changeset 1210:fa0800591306 feature/variable_derivatives

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Add note about removing apply_with_region
author Jonatan Werpers <jonatan@werpers.com>
date Fri, 03 Feb 2023 22:50:42 +0100
parents 703eaa3e50c4
children d2ffb412a0be 102ebdaf7c11
files src/LazyTensors/lazy_tensor_operations.jl
diffstat 1 files changed, 2 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/LazyTensors/lazy_tensor_operations.jl	Wed Apr 27 10:26:11 2022 +0200
+++ b/src/LazyTensors/lazy_tensor_operations.jl	Fri Feb 03 22:50:42 2023 +0100
@@ -316,7 +316,7 @@
     print(io, "range size $(range_size(err.tm)) of LazyTensor not matching size $(err.sz)")
 end
 
-
+# TODO: These should probably be removed. From new testing in performance/get_region_type_inference it seems that the problems are solved without these
 function apply_with_region(op, v, boundary_width::Integer, dim_size::Integer, i)
     if 0 < i <= boundary_width
         return LazyTensors.apply(op,v,Index(i,Lower))
@@ -340,4 +340,4 @@
     else
         error("Bounds error") # TODO: Make this more standard
     end
-end
\ No newline at end of file
+end