Mercurial > repos > public > sbplib_julia
comparison Notes.md @ 775:1d166e7a9deb refactor/sbp_operators_method_signatures
Add a note about the name of VolumeOperator
| author | Jonatan Werpers <jonatan@werpers.com> |
|---|---|
| date | Sat, 17 Jul 2021 17:50:18 +0200 |
| parents | d7d030f8f708 |
| children | 1de10f5b2008 |
comparison
equal
deleted
inserted
replaced
| 774:d8c0916853fc | 775:1d166e7a9deb |
|---|---|
| 328 Since the LazyOperators package is sort of implementing matrix-free matrices there is no concept of inner products there at the moment. It seems to complicate large parts of the package if this was included there. | 328 Since the LazyOperators package is sort of implementing matrix-free matrices there is no concept of inner products there at the moment. It seems to complicate large parts of the package if this was included there. |
| 329 | 329 |
| 330 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. | 330 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. |
| 331 | 331 |
| 332 | 332 |
| 333 ## Name of the `VolumeOperator` type for constant stencils | |
| 334 It seems that the name is too general. The name of the method `volume_operator` makes sense. It should return different types of `TensorMapping` specialized for the grid. A suggetion for a better name is `ConstantStencilVolumeOperator` |