Mercurial > repos > public > sbplib_julia

comparison TODO.md @ 626:8581610da4f1 feature/volume_and_boundary_operators

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Add comment for work on RegionIndices and fix typos
author Vidar Stiernström <vidar.stiernstrom@it.uu.se>
date Thu, 31 Dec 2020 08:11:57 +0100
parents 8f9b3eac128a
children 784920c7c9cd
comparison
equal deleted inserted replaced
625:1c512e796c6d 626:8581610da4f1
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 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. 10 - [ ] 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. 11 - [ ] 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)`) 12 - [ ] Replace getindex hack for flattening tuples with flatten_tuple. (eg. `getindex.(range_size.(L.D2),1)`)
13 - [ ] Use `@inferred` in a lot of tests. 13 - [ ] Use `@inferred` in a lot of tests.
14 - [ ] Make sure we are setting tolerances in tests in a consistent way 14 - [ ] 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 15 - [ ] 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 16 - [ ] 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 17 - [ ] Add boundschecking in TensorMappingApplication
18 - [ ] Start renaming things in LazyTensors 18 - [ ] Start renaming things in LazyTensors
19 - [ ] Clean up RegionIndices 19 - [ ] Clean up RegionIndices
20 1. [ ] Write tests for how things should work 20 1. [ ] Write tests for how things should work
21 2. [ ] Update RegionIndices accordingly 21 2. [ ] Update RegionIndices accordingly
22 3. [ ] Fix the rest of the library 22 3. [ ] Fix the rest of the library
23 - [ ] Add posibility to create tensor mapping application with `()`, e.g `D1(v) <=> D1*v`? 23 Should getregion also work for getregion(::Colon,...)
24 - [ ] Add possibility to create tensor mapping application with `()`, e.g `D1(v) <=> D1*v`?
24 25
25 ## Repo 26 ## Repo
26 - [ ] Add Vidar to the authors list 27 - [ ] Add Vidar to the authors list
27 - [ ] Rename repo to Sbplib.jl 28 - [ ] Rename repo to Sbplib.jl
28 29
29 # Wrap up tasks 30 # Wrap up tasks
30 - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen 31 - [ ] 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 32 - [ ] 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 33 - [ ] Kolla att vi har @inline på rätt ställen
33 - [ ] Profilera 34 - [ ] 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.