Mercurial > repos > public > sbplib_julia
annotate TODO.md @ 307:81e8968e222c
Update TODO
| author | Vidar Stiernström <vidar.stiernstrom@it.uu.se> |
|---|---|
| date | Wed, 09 Sep 2020 21:06:59 +0200 |
| parents | a0ec3cb6a65b |
| children | 0c8d4a734c4f |
| 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 | |
| 307 | 13 - [ ] Use traits for symmetric tensor mappings such that apply_transpoe = apply for all such mappings |
| 294 | 14 - [x] Move Laplace tensor operator to different package |
| 15 - [x] Remove grid as a property of the Laplace tensor operator | |
| 307 | 16 - [ ] Update how dependencies are handled for tests. This was updated in Julia v1.2 and would allow us to use test specific dev packages. |
| 119 | 17 |
| 294 | 18 ## Reasearch and thinking |
| 19 - [ ] Redo all Tensor applys to take Vararg instead of tuple of Index? | |
| 20 Have we been down that road before? Is there any reason not to do this? | |
| 21 - [ ] 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
|
22 - [ ] Check if Vidars design docs fit in there |
| 294 | 23 - [ ] Formalize how range_size() and domain_size() are supposed to work in TensorMappings where dim(domain) != dim(range) (add tests or document) |
| 24 - [x] Should there be some kind of collection struct for SBP operators (as TensorOperators), providing easy access to all parts (D2, e, d , -> YES! | |
| 25 H.. H_gamma etc.) | |
|
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
26 - [x] Is "missing" a good value for unknown dimension sizes (of `e*g` for example) |
| 307 | 27 - [ ] Create a macro @lazy which replaces a binary op (+,-) by its lazy equivalent? Would be a neat way to indicate which evaluations are lazy without cluttering/confusing with special characters. |
| 119 | 28 |
| 294 | 29 # Wrap up task |
|
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
30 |
| 294 | 31 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen |
| 32 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck | |
| 33 - [ ] Kolla att vi har @inline på rätt ställen | |
| 34 - [ ] Profilera | |
|
145
e0c8f5cf3a3f
Rename boundary indentifiers and use them inte the sat functions
Jonatan Werpers <jonatan@werpers.com>
parents:
119
diff
changeset
|
35 |
|
265
4308b500d6e7
Add ideas to TODO.txt
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
145
diff
changeset
|
36 |
| 294 | 37 # 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
|
38 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
|
39 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
|
40 |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
41 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
|
42 apply_quadrature? |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
43 |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
44 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
|
45 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
|
46 design |
|
284
0b8e041a1873
Change how range_size and domain_size work with BoundaryValues and NormalDerivative
Jonatan Werpers <jonatan@werpers.com>
parents:
274
diff
changeset
|
47 |
|
298
a0ec3cb6a65b
Add todo regarding traits and symmetric tensor mappings
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
297
diff
changeset
|
48 Specificera operatorer i TOML eller något liknande? |
