Mercurial > repos > public > sbplib_julia
comparison TODO.md @ 1365:4684c7f1c4cb feature/variable_derivatives
Merge with default
author | Vidar Stiernström <vidar.stiernstrom@it.uu.se> |
---|---|
date | Sun, 21 May 2023 21:55:14 +0200 |
parents | eeecdf135912 fa3695f634de |
children | 410bc321242c |
comparison
equal
deleted
inserted
replaced
1358:e7861cfb6ede | 1365:4684c7f1c4cb |
---|---|
1 # TODO | 1 # TODO |
2 | 2 |
3 ## Organization | |
4 - [ ] Split up Notes.md in several files | |
3 | 5 |
4 ## Coding | 6 ## Coding |
5 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide | 7 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide |
6 - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?) | 8 - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?) |
7 - [ ] Create a struct that bundles the necessary Tensor operators for solving the wave equation. | 9 - [ ] Create a struct that bundles the necessary Tensor operators for solving the wave equation. |
8 - [ ] Replace getindex hack for flattening tuples with flatten_tuple. (eg. `getindex.(range_size.(L.D2),1)`) | |
9 - [ ] Use `@inferred` in a lot of tests. | 10 - [ ] Use `@inferred` in a lot of tests. |
10 - [ ] Replace `@inferred` tests with a benchmark suite that automatically tests for regressions. | 11 - [ ] Replace `@inferred` tests with a benchmark suite that automatically tests for regressions. |
11 - [ ] Make sure we are setting tolerances in tests in a consistent way | 12 - [ ] Make sure we are setting tolerances in tests in a consistent way |
12 - [ ] Write down some coding guideline or checklist for code conventions. For example i,j,... for indices and I for multi-index | 13 - [ ] Write down some coding guideline or checklist for code conventions. For example i,j,... for indices and I for multi-index |
13 - [ ] Clean up RegionIndices | 14 - [ ] Clean up RegionIndices |
25 - [ ] Gå igenom alla typ parametrar och kolla om de är motiverade. Både i signaturer och typer, tex D i VariableSecondDerivative. Kan vi använda promote istället? | 26 - [ ] Gå igenom alla typ parametrar och kolla om de är motiverade. Både i signaturer och typer, tex D i VariableSecondDerivative. Kan vi använda promote istället? |
26 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen | 27 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen |
27 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck | 28 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck |
28 - [ ] Kolla att vi har @inline på rätt ställen | 29 - [ ] Kolla att vi har @inline på rätt ställen |
29 - [ ] Profilera | 30 - [ ] Profilera |
31 | |
32 | |
33 ### Grids | |
34 | |
35 - [ ] Multiblock grids | |
36 - [ ] Periodic grids | |
37 - [ ] Grids with modified boundary closures | |
38 | |
39 | |
40 ### Benchmarks | |
41 - [ ] Benchmarks for all grid indexing (focused on allocation) | |
42 - [ ] Benchmarks for indexing of lazy grid functions | |
43 - [ ] Add benchmarks for range type in EquidistantGrid. (LinRange vs StepRange) |