Mercurial > repos > public > sbplib_julia

comparison Notes.md @ 332:535f1bff4bcc

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Add some notes and some todos
author Jonatan Werpers <jonatan@werpers.com>
date Fri, 25 Sep 2020 11:33:41 +0200
parents d705b397aa33
children de4746d6d126
comparison
equal deleted inserted replaced
331:c8bbb4a83518 332:535f1bff4bcc
4 It might be a good idea let tensormappings know the size of their range and domain as a constant. This probably can't be enforced on the abstract type but maybe we should write our difference operators this way. Having this as default should clean up the thinking around adjoints of boundary operators. It could also simplify getting high performance out of repeated application of regioned TensorMappings. 4 It might be a good idea let tensormappings know the size of their range and domain as a constant. This probably can't be enforced on the abstract type but maybe we should write our difference operators this way. Having this as default should clean up the thinking around adjoints of boundary operators. It could also simplify getting high performance out of repeated application of regioned TensorMappings.
5 Is there any reason to use a trait to differentiate between fixed size and unknown size? 5 Is there any reason to use a trait to differentiate between fixed size and unknown size?
6 6
7 ## Test setup 7 ## Test setup
8 Once we figure out how to organize the subpackages we should update test folders to Project. As of writing this there seems to be and issue with this approach combined with dev'ed packages so we can't do it yet. It seems that Pkg might fix this in the future. 8 Once we figure out how to organize the subpackages we should update test folders to Project. As of writing this there seems to be and issue with this approach combined with dev'ed packages so we can't do it yet. It seems that Pkg might fix this in the future.
9
10 Some steps to imporve the situation right now is to combine everything to one package and use the `@includetests` macro from [TestSetExtensions](https://github.com/ssfrr/TestSetExtensions.jl) together with `Pkg.test(test_args="...")` to selectively run tests.
9 11
10 ## Reasearch and thinking 12 ## Reasearch and thinking
11 - [ ] Use a trait to indicate if a TensorMapping uses indices with regions. 13 - [ ] Use a trait to indicate if a TensorMapping uses indices with regions.
12 The default should be that they do NOT. 14 The default should be that they do NOT.
13 - [ ] What to name this trait? Can we call it IndexStyle but not export it to avoid conflicts with Base.IndexStyle? 15 - [ ] What to name this trait? Can we call it IndexStyle but not export it to avoid conflicts with Base.IndexStyle?
18 - [ ] Check if Vidars design docs fit in there 20 - [ ] Check if Vidars design docs fit in there
19 - [ ] Formalize how range_size() and domain_size() are supposed to work in TensorMappings where dim(domain) != dim(range) (add tests or document) 21 - [ ] Formalize how range_size() and domain_size() are supposed to work in TensorMappings where dim(domain) != dim(range) (add tests or document)
20 - [ ] Create a macro @lazy which replaces a binary op (+,-) by its lazy equivalent? Would be a neat way to indicate which evaluations are lazy without cluttering/confusing with special characters. 22 - [ ] Create a macro @lazy which replaces a binary op (+,-) by its lazy equivalent? Would be a neat way to indicate which evaluations are lazy without cluttering/confusing with special characters.
21 - [ ] Specificera operatorer i TOML eller något liknande? 23 - [ ] Specificera operatorer i TOML eller något liknande?
22 H.. H_gamma etc.) 24 H.. H_gamma etc.)
25 - [ ] Dispatch in Lower() instead of the type Lower so `::Lower` instead of `::Type{Lower}` ???
26 Seems better unless there is some specific reason to use the type instead of the value.