Mercurial > repos > public > sbplib_julia
annotate TODO.md @ 298:a0ec3cb6a65b
Add todo regarding traits and symmetric tensor mappings
| author | Vidar Stiernström <vidar.stiernstrom@it.uu.se> |
|---|---|
| date | Mon, 22 Jun 2020 23:22:08 +0200 |
| parents | dfadc0bafbbd |
| children | 81e8968e222c |
| rev | line source |
|---|---|
| 102 | 1 # TODO |
| 2 | |
| 294 | 3 ## Skämskudde |
| 4 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide | |
| 5 - [ ] Skriv tester | |
| 102 | 6 |
| 294 | 7 ## Coding |
|
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
8 - [ ] Add new Laplace opertor to DiffOps, probably named WaveEqOp(?!!?) |
| 294 | 9 - [ ] Add 1D operators (D1, D2, e, d ... ) as TensorOperators |
| 10 - [ ] Create a struct that bundles the necessary Tensor operators for solving the wave equation. | |
| 11 - [ ] Use traits like IndexStyle, IndexLinear, IndexCartesian to differentiate | |
| 12 TensorMappings that are flexible in size and those that are fixed in size | |
| 13 - [x] Move Laplace tensor operator to different package | |
| 14 - [x] Remove grid as a property of the Laplace tensor operator | |
| 119 | 15 |
| 294 | 16 ## Reasearch and thinking |
| 17 - [ ] Redo all Tensor applys to take Vararg instead of tuple of Index? | |
| 18 Have we been down that road before? Is there any reason not to do this? | |
| 19 - [ ] Check how the native julia doc generator works | |
|
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
20 - [ ] Check if Vidars design docs fit in there |
| 294 | 21 - [ ] Formalize how range_size() and domain_size() are supposed to work in TensorMappings where dim(domain) != dim(range) (add tests or document) |
| 22 - [x] Should there be some kind of collection struct for SBP operators (as TensorOperators), providing easy access to all parts (D2, e, d , -> YES! | |
| 23 H.. H_gamma etc.) | |
|
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
24 - [x] Is "missing" a good value for unknown dimension sizes (of `e*g` for example) |
|
298
a0ec3cb6a65b
Add todo regarding traits and symmetric tensor mappings
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
297
diff
changeset
|
25 - [] Add traits for symmetric tensor mappings such that apply_transpoe = apply for all such mappings |
| 119 | 26 |
| 294 | 27 # Wrap up task |
|
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
28 |
| 294 | 29 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen |
| 30 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck | |
| 31 - [ ] Kolla att vi har @inline på rätt ställen | |
| 32 - [ ] Profilera | |
|
145
e0c8f5cf3a3f
Rename boundary indentifiers and use them inte the sat functions
Jonatan Werpers <jonatan@werpers.com>
parents:
119
diff
changeset
|
33 |
|
265
4308b500d6e7
Add ideas to TODO.txt
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
145
diff
changeset
|
34 |
| 294 | 35 # Old stuff todos (Are these still relevant?) |
|
274
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
36 Borde det finns motsvarande apply_stencil för apply_quadrature, |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
37 apply_boundary_value och apply_normal_derivative? |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
38 |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
39 Borde man alltid skicka in N som parameter i apply_2nd_derivative, t.ex som i |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
40 apply_quadrature? |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
41 |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
42 Just nu agerar apply_normal_derivative, apply_boundary_value på inte på v som |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
43 en vektor, utan randvärdet plockas ut utanför. Känns inte konsistent med övrig |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
44 design |
|
284
0b8e041a1873
Change how range_size and domain_size work with BoundaryValues and NormalDerivative
Jonatan Werpers <jonatan@werpers.com>
parents:
274
diff
changeset
|
45 |
|
298
a0ec3cb6a65b
Add todo regarding traits and symmetric tensor mappings
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
297
diff
changeset
|
46 Specificera operatorer i TOML eller något liknande? |
