changeset 914:a9d776400251 feature/variable_derivatives

Clean up TODO.md
author Jonatan Werpers <jonatan@werpers.com>
date Fri, 18 Feb 2022 07:18:38 +0100
parents cad3a9f82009
children 50d2c2b9dc2c
files TODO.md
diffstat 1 files changed, 4 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/TODO.md	Thu Feb 17 08:46:19 2022 +0100
+++ b/TODO.md	Fri Feb 18 07:18:38 2022 +0100
@@ -1,15 +1,12 @@
 # TODO
 
-## Skämskudde
+
  - [ ] Ändra namn på variabler och funktioner så att det följer style-guide
- - [ ] Skriv tester
-
-## Coding
  - [ ] Add new Laplace operator to DiffOps, probably named WaveEqOp(?!!?)
  - [ ] Create a struct that bundles the necessary Tensor operators for solving the wave equation.
- - [ ] Add a quick and simple way of running all tests for all subpackages.
  - [ ] Replace getindex hack for flattening tuples with flatten_tuple. (eg. `getindex.(range_size.(L.D2),1)`)
  - [ ] Use `@inferred` in a lot of tests.
+ - [ ] Replace `@inferred` tests with a benchmark suite that automatically tests for regressions.
  - [ ] Make sure we are setting tolerances in tests in a consistent way
  - [ ] Add check for correct domain sizes to lazy tensor operations using SizeMismatch
  - [ ] Write down some coding guideline or checklist for code conventions. For example i,j,... for indices and I for multi-index
@@ -23,12 +20,10 @@
  - [ ] Add possibility to create tensor mapping application with `()`, e.g `D1(v) <=> D1*v`?
  - [ ] Add custom pretty printing to LazyTensors/SbpOperators to enhance readability of e.g error messages.
        See (https://docs.julialang.org/en/v1/manual/types/#man-custom-pretty-printing)
- - [ ] Gå igenom alla typ parametrar och kolla om de är motiverade. Både i signaturer och typer, tex D i VariableSecondDerivative
+ - [ ] Samla noggrannhets- och SBP-ness-tester för alla operatorer på ett ställe
 
-## Repo
- - [ ] Rename repo to Sbplib.jl
 
-# Wrap up tasks
+ - [ ] 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?
  - [ ] Kolla att vi har @inbounds och @propagate_inbounds på rätt ställen
  - [ ] Kolla att vi gör boundschecks överallt och att de är markerade med @boundscheck
  - [ ] Kolla att vi har @inline på rätt ställen