Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Definition of Ko in chrono #107

Open
yumengch opened this issue Oct 6, 2023 · 2 comments
Open

Definition of Ko in chrono #107

yumengch opened this issue Oct 6, 2023 · 2 comments

Comments

@yumengch
Copy link
Contributor

yumengch commented Oct 6, 2023

Hi Patrick

I understand this might be trivial but I feel this can help others better understand the time handling in DAPPER. I find the definition of HMM.tseq.Ko a bit confusing.

I can understand that the whole trajectory is K+1 with K being the number of forecast time steps (len(kk) == len(tt) == K + 1), and 1 is the initial condition.

But the logic of having len(kko) == len(tto) == Ko+1 is unclear to me. The only reason that I can think of is to match the definition of K. However, we don't have any observation values at initial step, so in the end, when I try to describe Ko, it is always the number of observations minus one. I feel this is an unconventional concept.

@patnr
Copy link
Collaborator

patnr commented Oct 6, 2023

Hi! I think indeed there is something suboptimal about this convention. I made the choice about it very early on, when I was still relatively inexperienced with Python vs Matlab. I'm a bit busy now, but will try to say something more soon!

@patnr
Copy link
Collaborator

patnr commented Oct 6, 2023

Yes, I completely agree with you. At the time of making the choice I was too amateur with python and felt that kko[Ko] (rather than kko[-1]) would be valuable.

PR would be welcome. Grepping for Ko turns up 82 counts, but there might be other more subtle/implicit hardcoded dependencies on the convention, perhaps necessitating further grepping for len.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants