Mercurial > repos > public > sbplib_julia
comparison Notes.md @ 1224:e2f6dafb5d83 refactor/LazyTensors/tuple_manipulation
Add note about LazyOuterProduct
| author | Jonatan Werpers <jonatan@werpers.com> |
|---|---|
| date | Sat, 18 Feb 2023 12:18:34 +0100 |
| parents | 396278072f18 |
| children | a8fa8c1137cc |
comparison
equal
deleted
inserted
replaced
| 1223:5bfb182e24dc | 1224:e2f6dafb5d83 |
|---|---|
| 385 A different approach would be to include it as a trait for operators so that you can specify what the adjoint for that operator is. | 385 A different approach would be to include it as a trait for operators so that you can specify what the adjoint for that operator is. |
| 386 | 386 |
| 387 | 387 |
| 388 ## Name of the `VolumeOperator` type for constant stencils | 388 ## Name of the `VolumeOperator` type for constant stencils |
| 389 It seems that the name is too general. The name of the method `volume_operator` makes sense. It should return different types of `LazyTensor` specialized for the grid. A suggetion for a better name is `ConstantStencilVolumeOperator` | 389 It seems that the name is too general. The name of the method `volume_operator` makes sense. It should return different types of `LazyTensor` specialized for the grid. A suggetion for a better name is `ConstantStencilVolumeOperator` |
| 390 | |
| 391 | |
| 392 ## Implementation of LazyOuterProduct | |
| 393 Could the implementation of LazyOuterProduct be simplified by making it a | |
| 394 struct containing two or more LazyTensors? (using split_tuple in a similar way | |
| 395 as TensorGrid) |