comparison TODO.md @ 1050:396278072f18

Add some notes and clean up TODO.md a little bit
author Jonatan Werpers <jonatan@werpers.com>
date Wed, 23 Mar 2022 13:09:31 +0100
parents 6b94663f4376
children eeecdf135912 6567e38b05ca
comparison
equal deleted inserted replaced
1043:c16116e403e2 1050:396278072f18
1 # TODO 1 # TODO
2 2
3 ## Skämskudde
4 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide
5 - [ ] Skriv tester
6 3
7 ## Coding 4 ## Coding
5 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide
8 - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?) 6 - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?)
9 - [ ] 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.
10 - [ ] Replace getindex hack for flattening tuples with flatten_tuple. (eg. `getindex.(range_size.(L.D2),1)`) 8 - [ ] Replace getindex hack for flattening tuples with flatten_tuple. (eg. `getindex.(range_size.(L.D2),1)`)
11 - [ ] Use `@inferred` in a lot of tests. 9 - [ ] Use `@inferred` in a lot of tests.
10 - [ ] Replace `@inferred` tests with a benchmark suite that automatically tests for regressions.
12 - [ ] Make sure we are setting tolerances in tests in a consistent way 11 - [ ] Make sure we are setting tolerances in tests in a consistent way
13 - [ ] Write down some coding guideline or checklist for code conventions. For example i,j,... for indices and I for multi-index 12 - [ ] Write down some coding guideline or checklist for code conventions. For example i,j,... for indices and I for multi-index
14 - [ ] Clean up RegionIndices 13 - [ ] Clean up RegionIndices
15 1. [ ] Write tests for how things should work 14 1. [ ] Write tests for how things should work
16 2. [ ] Update RegionIndices accordingly 15 2. [ ] Update RegionIndices accordingly
17 3. [ ] Fix the rest of the library 16 3. [ ] Fix the rest of the library
18 Should getregion also work for getregion(::Colon,...) 17 Should getregion also work for getregion(::Colon,...)
19 - [ ] Add possibility to create tensor mapping application with `()`, e.g `D1(v) <=> D1*v`? 18 - [ ] Add possibility to create tensor mapping application with `()`, e.g `D1(v) <=> D1*v`?
20 - [ ] Add custom pretty printing to LazyTensors/SbpOperators to enhance readability of e.g error messages. 19 - [ ] Add custom pretty printing to LazyTensors/SbpOperators to enhance readability of e.g error messages.
21 See (https://docs.julialang.org/en/v1/manual/types/#man-custom-pretty-printing) 20 See (https://docs.julialang.org/en/v1/manual/types/#man-custom-pretty-printing)
21 - [ ] Samla noggrannhets- och SBP-ness-tester för alla operatorer på ett ställe
22 - [ ] Move export statements to top of each module 22 - [ ] Move export statements to top of each module
23 23
24 ## Repo
25 - [ ] Rename repo to Sbplib.jl
26 24
27 # Wrap up tasks 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?
28 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen 26 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen
29 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck 27 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck
30 - [ ] Kolla att vi har @inline på rätt ställen 28 - [ ] Kolla att vi har @inline på rätt ställen
31 - [ ] Profilera 29 - [ ] Profilera