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)