comparison TODO.md @ 1225:6567e38b05ca refactor/LazyTensors/tuple_manipulation

Replace flatten_tuple with concatenate_tuple
author Jonatan Werpers <jonatan@werpers.com>
date Sat, 18 Feb 2023 12:24:44 +0100
parents 396278072f18
children c0c1189c5f2e
comparison
equal deleted inserted replaced
1224:e2f6dafb5d83 1225:6567e38b05ca
3 3
4 ## Coding 4 ## Coding
5 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide 5 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide
6 - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?) 6 - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?)
7 - [ ] Create a struct that bundles the necessary Tensor operators for solving the wave equation. 7 - [ ] 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. 8 - [ ] Use `@inferred` in a lot of tests.
10 - [ ] Replace `@inferred` tests with a benchmark suite that automatically tests for regressions. 9 - [ ] 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 10 - [ ] 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 11 - [ ] 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 12 - [ ] Clean up RegionIndices