comparison Notes.md @ 939:d8da3a1060b7 feature/variable_derivatives

Add note about where to put Regions
author Jonatan Werpers <jonatan@werpers.com>
date Wed, 09 Mar 2022 10:14:16 +0100
parents eb03bda76bae
children f885e1de6dc4
comparison
equal deleted inserted replaced
937:025a506ca2fa 939:d8da3a1060b7
146 Seems better unless there is some specific reason to use the type instead of the value. 146 Seems better unless there is some specific reason to use the type instead of the value.
147 - [ ] How do we handle mixes of periodic and non-periodic grids? Seems it should be supported on the grid level and on the 1d operator level. Between there it should be transparent. 147 - [ ] How do we handle mixes of periodic and non-periodic grids? Seems it should be supported on the grid level and on the 1d operator level. Between there it should be transparent.
148 - [ ] Can we have a trait to tell if a TensorMapping is transposable? 148 - [ ] Can we have a trait to tell if a TensorMapping is transposable?
149 - [ ] Is it ok to have "Constructors" for abstract types which create subtypes? For example a Grids() functions that gives different kind of grids based on input? 149 - [ ] Is it ok to have "Constructors" for abstract types which create subtypes? For example a Grids() functions that gives different kind of grids based on input?
150 150
151 ## Identifiers for regions
152 The identifiers (`Upper`, `Lower`, `Interior`) used for region indecies should probabily be included in the grid module. This allows new grid types to come with their own regions.
153
151 ## Regions and tensormappings 154 ## Regions and tensormappings
152 - [ ] Use a trait to indicate if a TensorMapping uses indices with regions. 155 - [ ] Use a trait to indicate if a TensorMapping uses indices with regions.
153 The default should be that they do NOT. 156 The default should be that they do NOT.
154 - [ ] What to name this trait? Can we call it IndexStyle but not export it to avoid conflicts with Base.IndexStyle? 157 - [ ] What to name this trait? Can we call it IndexStyle but not export it to avoid conflicts with Base.IndexStyle?
155 - [ ] Figure out repeated application of regioned TensorMappings. Maybe an instance of a tensor mapping needs to know the exact size of the range and domain for this to work? 158 - [ ] Figure out repeated application of regioned TensorMappings. Maybe an instance of a tensor mapping needs to know the exact size of the range and domain for this to work?