Mercurial > repos > public > sbplib_julia

comparison TODO.md @ 651:67639b1c99ea

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Merged feature/volume_and_boundary_operators
author Vidar Stiernström <vidar.stiernstrom@it.uu.se>
date Wed, 20 Jan 2021 17:52:55 +0100
parents 784920c7c9cd
children 2a0b17adaf9e
comparison
equal deleted inserted replaced
615:52749b687a67 651:67639b1c99ea
3 ## Skämskudde 3 ## Skämskudde
4 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide 4 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide
5 - [ ] Skriv tester 5 - [ ] Skriv tester
6 6
7 ## Coding 7 ## Coding
8 - [ ] Add new Laplace opertor to DiffOps, probably named WaveEqOp(?!!?) 8 - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?)
9 - [ ] Add 1D operators (D1, D2, e, d ... ) as TensorOperators
10 - [ ] 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.
11 - [ ] Add a quick and simple way of running all tests for all subpackages. 10 - [ ] Add a quick and simple way of running all tests for all subpackages.
12 - [ ] Replace getindex hack for flatteing tuples with flatten_tuple. (eg. `getindex.(range_size.(L.D2),1)`) 11 - [ ] Replace getindex hack for flattening tuples with flatten_tuple. (eg. `getindex.(range_size.(L.D2),1)`)
13 - [ ] Use `@inferred` in a lot of tests. 12 - [ ] Use `@inferred` in a lot of tests.
14 - [ ] Make sure we are setting tolerances in tests in a consistent way 13 - [ ] Make sure we are setting tolerances in tests in a consistent way
15 - [ ] Add check for correct domain sizes to lazy tensor operations using SizeMismatch 14 - [ ] Add check for correct domain sizes to lazy tensor operations using SizeMismatch
16 - [ ] Write down some coding guideline or checklist for code convetions. For example i,j,... for indecies and I for multi-index 15 - [ ] Write down some coding guideline or checklist for code conventions. For example i,j,... for indices and I for multi-index
17 - [ ] Add boundschecking in TensorMappingApplication 16 - [ ] Add boundschecking in TensorMappingApplication
18 - [ ] Start renaming things in LazyTensors 17 - [ ] Start renaming things in LazyTensors
19 - [ ] Clean up RegionIndices 18 - [ ] Clean up RegionIndices
20 1. [ ] Write tests for how things should work 19 1. [ ] Write tests for how things should work
21 2. [ ] Update RegionIndices accordingly 20 2. [ ] Update RegionIndices accordingly
22 3. [ ] Fix the rest of the library 21 3. [ ] Fix the rest of the library
23 - [ ] Add posibility to create tensor mapping application with `()`, e.g `D1(v) <=> D1*v`? 22 Should getregion also work for getregion(::Colon,...)
23 - [ ] Add possibility to create tensor mapping application with `()`, e.g `D1(v) <=> D1*v`?
24 24
25 ## Repo 25 ## Repo
26 - [ ] Add Vidar to the authors list 26 - [ ] Add Vidar to the authors list
27 - [ ] Rename repo to Sbplib.jl 27 - [ ] Rename repo to Sbplib.jl
28 28
29 # Wrap up tasks 29 # Wrap up tasks
30 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen 30 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen
31 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck 31 - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck
32 - [ ] Kolla att vi har @inline på rätt ställen 32 - [ ] Kolla att vi har @inline på rätt ställen
33 - [ ] Profilera 33 - [ ] Profilera
34
35
36 # Old stuff todos (Are these still relevant?)
37 Borde det finns motsvarande apply_stencil för apply_quadrature,
38 apply_boundary_value och apply_normal_derivative?
39
40 Borde man alltid skicka in N som parameter i apply_2nd_derivative, t.ex som i
41 apply_quadrature?
42
43 Just nu agerar apply_normal_derivative, apply_boundary_value på inte på v som
44 en vektor, utan randvärdet plockas ut utanför. Känns inte konsistent med övrig design.