Mercurial > repos > public > sbplib_julia
comparison TODO.md @ 1101:1e8270c18edb feature/lazy_tensors/pretty_printing
Merge default
author | Jonatan Werpers <jonatan@werpers.com> |
---|---|
date | Thu, 12 May 2022 21:52:47 +0200 |
parents | 396278072f18 |
children | eeecdf135912 6567e38b05ca |
comparison
equal
deleted
inserted
replaced
1014:67969bd7e642 | 1101:1e8270c18edb |
---|---|
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 - [ ] Add check for correct domain sizes to lazy tensor operations using SizeMismatch | |
14 - [ ] 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 |
15 - [ ] Add boundschecking in LazyTensorApplication | |
16 - [ ] Start renaming things in LazyTensors | |
17 - [ ] Clean up RegionIndices | 13 - [ ] Clean up RegionIndices |
18 1. [ ] Write tests for how things should work | 14 1. [ ] Write tests for how things should work |
19 2. [ ] Update RegionIndices accordingly | 15 2. [ ] Update RegionIndices accordingly |
20 3. [ ] Fix the rest of the library | 16 3. [ ] Fix the rest of the library |
21 Should getregion also work for getregion(::Colon,...) | 17 Should getregion also work for getregion(::Colon,...) |
22 - [ ] 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`? |
23 - [ ] 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. |
24 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 | |
25 - [ ] Move export statements to top of each module | 22 - [ ] Move export statements to top of each module |
26 - [ ] Add a type StencilSet for easier dispatch | |
27 | 23 |
28 ## Repo | |
29 - [ ] Rename repo to Sbplib.jl | |
30 | 24 |
31 # 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? |
32 - [ ] 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 |
33 - [ ] 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 |
34 - [ ] Kolla att vi har @inline på rätt ställen | 28 - [ ] Kolla att vi har @inline på rätt ställen |
35 - [ ] Profilera | 29 - [ ] Profilera |