Closed rob-luke closed 9 years ago
This looks interesting. cc @scidom
Yeap, that's a good idea - we may have optional or named arguments for G
in StateSpaceModel
to define default behaviour. How do you propose we set the default to behaviour, time-variant or invariant matrices?
Do you want to open a PR?
Glad people think its interesting. I think the current PR works with either time variant or time invariant matrices, what do you propose I should change?
Also is there a test suite available? I was a little worried id slip up on the i
and i-1
s.
Ive also not optimised for speed or multiple types, this was just a first pass to see if there was interest (which it appears there is :) ).
Well, I must admit that we need to set up the test suite and also do a couple of nice changes in the interface we discussed with @milktrader a while ago regarding the type definitions of models. I need to put it in my todo list.
Yes, if you can write up a test, without worrying too much for its placement or name but mostly that it is operational, it would be helpful.
Great, any optimisations are welcome. Only one quick question, how is the time invariant case accounted for? As I can see G
is time-dependent in the example above.
Okay, I'll put the test suite back on my radar. I'll update travis.yml and get us up to speed on the latest best practices.
@scidom No problem, I can add a few tests. Ill have time before the end of the week.
@milktrader I would also be keen to hear about test suite best practices. Is there a standard for commenting/documentation yet? Ive been using Docile.j
Thanks @codles, appreciated. Once you add the test(s), we are ready to merge. Thanks for the PR.
I'm thinking to go the readthedocs route for online documentation and just started playing with the new @doc
macro that v0.4
just implemented for REPL documentation (supporting ?
in REPL). I'll look at Docile.jl. Here is a link about the new macro discussion.https://github.com/JuliaLang/julia/issues/8966
Oh I see. Docile.jl is being integrated into Base. So we're talking about the same thing.
It can be useful to have the system matrices vary with time. Is this within the scope of this package?
This is an initial implementation to open discussion. With a modified version of the oldkalman test