Mercurial > repos > public > sbplib_julia
view TODO.md @ 355:5c9212a8ee4f refactor/remove_dynamic_size_tensormapping
Merge in subgrid function
author | Jonatan Werpers <jonatan@werpers.com> |
---|---|
date | Sun, 27 Sep 2020 21:08:18 +0200 |
parents | 535f1bff4bcc |
children | de4746d6d126 |
line wrap: on
line source
# TODO ## Skämskudde - [ ] Ändra namn på variabler och funktioner så att det följer style-guide - [ ] Skriv tester ## Coding - [ ] Add new Laplace opertor to DiffOps, probably named WaveEqOp(?!!?) - [ ] Add 1D operators (D1, D2, e, d ... ) as TensorOperators - [ ] 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. ## Other - [ ] Add Vidar to the authors list # Wrap up tasks - [ ] 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 - [ ] Profilera # Old stuff todos (Are these still relevant?) Borde det finns motsvarande apply_stencil för apply_quadrature, apply_boundary_value och apply_normal_derivative? Borde man alltid skicka in N som parameter i apply_2nd_derivative, t.ex som i apply_quadrature? Just nu agerar apply_normal_derivative, apply_boundary_value på inte på v som en vektor, utan randvärdet plockas ut utanför. Känns inte konsistent med övrig design.