Mercurial > repos > public > sbplib_julia
annotate TODO.md @ 427:1c41f4fd3e61
Add some notes and todos
author | Jonatan Werpers <jonatan@werpers.com> |
---|---|
date | Mon, 19 Oct 2020 10:28:21 +0200 |
parents | de4746d6d126 |
children | 7e698030c170 |
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. | |
320
d705b397aa33
Clean up Notes.md and TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
319
diff
changeset
|
11 - [ ] Add a quick and simple way of running all tests for all subpackages. |
379
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
12 - [ ] Replace getindex hack for flatteing tuples with flatten_tuple. |
427
1c41f4fd3e61
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
379
diff
changeset
|
13 - [ ] Fix indexing signatures. We should make sure we are not too specific. For the "inbetween" layers we don't know what type of index is coming so we should use `I...` instead of `I::Vararg{Int,R}` or probably better `I::Vararg{Any,R}` |
379
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
14 - [ ] Use `@inferred` in a lot of tests. |
427
1c41f4fd3e61
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
379
diff
changeset
|
15 - [ ] Make sure we are setting tolerances in tests in a consistent way |
119 | 16 |
379
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
17 ## Repo |
332
535f1bff4bcc
Add some notes and some todos
Jonatan Werpers <jonatan@werpers.com>
parents:
320
diff
changeset
|
18 - [ ] Add Vidar to the authors list |
379
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
19 - [ ] Rename repo to Sbplib.jl |
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
20 |
320
d705b397aa33
Clean up Notes.md and TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
319
diff
changeset
|
21 # Wrap up tasks |
294 | 22 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen |
23 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck | |
24 - [ ] Kolla att vi har @inline på rätt ställen | |
25 - [ ] Profilera | |
145
e0c8f5cf3a3f
Rename boundary indentifiers and use them inte the sat functions
Jonatan Werpers <jonatan@werpers.com>
parents:
119
diff
changeset
|
26 |
265
4308b500d6e7
Add ideas to TODO.txt
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
145
diff
changeset
|
27 |
294 | 28 # 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
|
29 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
|
30 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
|
31 |
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
32 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
|
33 apply_quadrature? |
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
34 |
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
35 Just nu agerar apply_normal_derivative, apply_boundary_value på inte på v som |
320
d705b397aa33
Clean up Notes.md and TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
319
diff
changeset
|
36 en vektor, utan randvärdet plockas ut utanför. Känns inte konsistent med övrig design. |