Mercurial > repos > public > sbplib_julia
annotate TODO.md @ 549:ace46ca8513f refactor/tensor_index_coupling
Remove completed todo
| author | Jonatan Werpers <jonatan@werpers.com> |
|---|---|
| date | Fri, 27 Nov 2020 17:53:26 +0100 |
| parents | 011ca1639153 |
| children | 8f9b3eac128a |
| 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. |
|
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
13 - [ ] Use `@inferred` in a lot of tests. |
|
427
1c41f4fd3e61
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
379
diff
changeset
|
14 - [ ] Make sure we are setting tolerances in tests in a consistent way |
| 490 | 15 - [ ] Add check for correct domain sizes to lazy tensor operations using SizeMismatch |
|
542
011ca1639153
Remove Index{Unknown} and replace with general Any implementations
Jonatan Werpers <jonatan@werpers.com>
parents:
490
diff
changeset
|
16 - [ ] Write down some coding guideline or checklist for code convetions. For example i,j,... för indecies and I for multi-index |
| 119 | 17 |
|
379
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
18 ## Repo |
|
332
535f1bff4bcc
Add some notes and some todos
Jonatan Werpers <jonatan@werpers.com>
parents:
320
diff
changeset
|
19 - [ ] Add Vidar to the authors list |
|
379
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
20 - [ ] Rename repo to Sbplib.jl |
|
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
21 |
|
320
d705b397aa33
Clean up Notes.md and TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
319
diff
changeset
|
22 # Wrap up tasks |
| 294 | 23 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen |
| 24 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck | |
| 25 - [ ] Kolla att vi har @inline på rätt ställen | |
| 26 - [ ] Profilera | |
|
145
e0c8f5cf3a3f
Rename boundary indentifiers and use them inte the sat functions
Jonatan Werpers <jonatan@werpers.com>
parents:
119
diff
changeset
|
27 |
|
265
4308b500d6e7
Add ideas to TODO.txt
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
145
diff
changeset
|
28 |
| 294 | 29 # 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
|
30 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
|
31 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
|
32 |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
33 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
|
34 apply_quadrature? |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
35 |
|
11010bb74260
Dispatch getindex for TensorMappingApplication on region indices. Dispatch apply
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
265
diff
changeset
|
36 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
|
37 en vektor, utan randvärdet plockas ut utanför. Känns inte konsistent med övrig design. |
