Mercurial > repos > public > sbplib_julia
annotate TODO.md @ 1004:7fd37aab84fe refactor/lazy_tensors
Simplify bounds handling for LazyElementwiseOperation
author | Jonatan Werpers <jonatan@werpers.com> |
---|---|
date | Sun, 20 Mar 2022 21:35:20 +0100 |
parents | 1ba8a398af9c |
children | 6abbb2c6c3e4 |
rev | line source |
---|---|
102 | 1 # TODO |
2 | |
294 | 3 ## Skämskudde |
4 - [ ] Ändra namn på variabler och funktioner så att det följer style-guide | |
5 - [ ] Skriv tester | |
102 | 6 |
294 | 7 ## Coding |
626
8581610da4f1
Add comment for work on RegionIndices and fix typos
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
606
diff
changeset
|
8 - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?) |
294 | 9 - [ ] Create a struct that bundles the necessary Tensor operators for solving the wave equation. |
626
8581610da4f1
Add comment for work on RegionIndices and fix typos
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
606
diff
changeset
|
10 - [ ] Replace getindex hack for flattening tuples with flatten_tuple. (eg. `getindex.(range_size.(L.D2),1)`) |
379
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
11 - [ ] Use `@inferred` in a lot of tests. |
427
1c41f4fd3e61
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
379
diff
changeset
|
12 - [ ] Make sure we are setting tolerances in tests in a consistent way |
490 | 13 - [ ] Add check for correct domain sizes to lazy tensor operations using SizeMismatch |
626
8581610da4f1
Add comment for work on RegionIndices and fix typos
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
606
diff
changeset
|
14 - [ ] Write down some coding guideline or checklist for code conventions. For example i,j,... for indices and I for multi-index |
995 | 15 - [ ] Add boundschecking in LazyTensorApplication |
606
8f9b3eac128a
Add a bunch of stuff to TODO.md and Notes.md
Jonatan Werpers <jonatan@werpers.com>
parents:
549
diff
changeset
|
16 - [ ] Start renaming things in LazyTensors |
8f9b3eac128a
Add a bunch of stuff to TODO.md and Notes.md
Jonatan Werpers <jonatan@werpers.com>
parents:
549
diff
changeset
|
17 - [ ] Clean up RegionIndices |
8f9b3eac128a
Add a bunch of stuff to TODO.md and Notes.md
Jonatan Werpers <jonatan@werpers.com>
parents:
549
diff
changeset
|
18 1. [ ] Write tests for how things should work |
8f9b3eac128a
Add a bunch of stuff to TODO.md and Notes.md
Jonatan Werpers <jonatan@werpers.com>
parents:
549
diff
changeset
|
19 2. [ ] Update RegionIndices accordingly |
8f9b3eac128a
Add a bunch of stuff to TODO.md and Notes.md
Jonatan Werpers <jonatan@werpers.com>
parents:
549
diff
changeset
|
20 3. [ ] Fix the rest of the library |
626
8581610da4f1
Add comment for work on RegionIndices and fix typos
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
606
diff
changeset
|
21 Should getregion also work for getregion(::Colon,...) |
8581610da4f1
Add comment for work on RegionIndices and fix typos
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
606
diff
changeset
|
22 - [ ] Add possibility to create tensor mapping application with `()`, e.g `D1(v) <=> D1*v`? |
656
1c3378b26466
Add todo on pretty printing
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
653
diff
changeset
|
23 - [ ] Add custom pretty printing to LazyTensors/SbpOperators to enhance readability of e.g error messages. |
1c3378b26466
Add todo on pretty printing
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
653
diff
changeset
|
24 See (https://docs.julialang.org/en/v1/manual/types/#man-custom-pretty-printing) |
963
52a0ebad0778
Review: Address review comment on placement of exports. Export statements should be placed at the top of the module file. Add todo.
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
656
diff
changeset
|
25 - [ ] Move export statements to top of each module |
966
365bc4e2a6a2
Add comment and todo on introducing a type StencilSet for dispatch
Vidar Stiernström <vidar.stiernstrom@it.uu.se>
parents:
963
diff
changeset
|
26 - [ ] Add a type StencilSet for easier dispatch |
119 | 27 |
379
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
28 ## Repo |
de4746d6d126
Add some notes and todos
Jonatan Werpers <jonatan@werpers.com>
parents:
332
diff
changeset
|
29 - [ ] Rename repo to Sbplib.jl |
297
dfadc0bafbbd
Fix some formatting errors in TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
295
diff
changeset
|
30 |
320
d705b397aa33
Clean up Notes.md and TODO.md
Jonatan Werpers <jonatan@werpers.com>
parents:
319
diff
changeset
|
31 # Wrap up tasks |
294 | 32 - [ ] 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 | |
34 - [ ] Kolla att vi har @inline på rätt ställen | |
35 - [ ] Profilera |