Mercurial > repos > public > sbplib_julia
comparison TODO.txt @ 294:9c12d9eb38fd
Clean up TODO.txt
author | Jonatan Werpers <jonatan@werpers.com> |
---|---|
date | Mon, 22 Jun 2020 22:17:33 +0200 |
parents | 0b8e041a1873 |
children |
comparison
equal
deleted
inserted
replaced
291:0f94dc29c4bf | 294:9c12d9eb38fd |
---|---|
1 # TODO | 1 # TODO |
2 | 2 |
3 Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen | 3 ## Skämskudde |
4 Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck | 4 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide |
5 Kolla att vi har @inline på rätt ställen | 5 - [ ] Skriv tester |
6 | 6 |
7 Ändra namn på variabler och funktioner så att det följer style-guide | 7 ## Coding |
8 * Add new Laplace opertor to DiffOps, probably named WaveEqOp(?!!?) | |
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 | |
8 | 15 |
9 Profilera | 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 | |
20 * Check if Vidars design docs fit in there | |
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.) | |
24 - [x] Is "missing" a good value for unknown dimension sizes (of e*g for example) | |
10 | 25 |
11 Skriv tester | 26 # Wrap up task |
27 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen | |
28 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck | |
29 - [ ] Kolla att vi har @inline på rätt ställen | |
30 - [ ] Profilera | |
12 | 31 |
13 Specificera operatorer i TOML eller något liknande? | |
14 | 32 |
33 # Old stuff todos (Are these still relevant?) | |
15 Borde det finns motsvarande apply_stencil för apply_quadrature, | 34 Borde det finns motsvarande apply_stencil för apply_quadrature, |
16 apply_boundary_value och apply_normal_derivative? | 35 apply_boundary_value och apply_normal_derivative? |
17 | 36 |
18 Borde man alltid skicka in N som parameter i apply_2nd_derivative, t.ex som i | 37 Borde man alltid skicka in N som parameter i apply_2nd_derivative, t.ex som i |
19 apply_quadrature? | 38 apply_quadrature? |
20 | 39 |
21 Just nu agerar apply_normal_derivative, apply_boundary_value på inte på v som | 40 Just nu agerar apply_normal_derivative, apply_boundary_value på inte på v som |
22 en vektor, utan randvärdet plockas ut utanför. Känns inte konsistent med övrig | 41 en vektor, utan randvärdet plockas ut utanför. Känns inte konsistent med övrig |
23 design | 42 design |
24 | 43 |
25 ## TODO 2020-06-18 | 44 Specificera operatorer i TOML eller något liknande? |
26 * Remove grid as a property of the Laplace tensor operator | |
27 * Add 1D operators (D1, D2, e, d ... ) as TensorOperators | |
28 * Move Laplace tensor operator to different package | |
29 * Add new Laplace opertor to DiffOps, probably named WaveEqOp(?!!?) | |
30 * Decide: Should there be some kind of collection struct for SBP operators (as TensorOperators), providing easy access to all parts (D2, e, d , | |
31 H.. H_gamma etc.) | |
32 * Is "missing" a good value for unknown dimension sizes (of e*g for example) | |
33 * Formalize how range_size() and domain_size() are supposed to work in TensorMappings where dim(domain) != dim(range) (add tests or document) |