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)