-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Add function fit_discrete_mc #678
Comments
One thing I am concerned with is that it is less general than what the general name
If this works, we can proceed for now as you propose. |
Thanks for your comments @oyamad . The extension you suggest sounds like a good idea. My understanding is that someone can proceed now in line with my suggestion and your bullet point one. We can then open as issue to add the enhancement that you suggested. |
Hi @Smit-create , yes, that's all we need. The same lines appear in 745c0d4. Notice that there's also an Many thanks! |
See this part: QuantEcon.py/quantecon/markov/approximation.py Lines 367 to 375 in 745c0d4
|
Done by #681 |
As discussed in #640, we should add a function called$X_0, \ldots, X_{T-1}$ taking values in $\mathbb R^n$ (each $X_t$ is a vector in continuous state space $\mathbb R^n$ ) and fits a finite Markov chain to it using the following steps:
fit_discrete_mc
that takes a time seriesMarkovChain
object.This function just combines existing tools, since
#. for step 3, we can use
cartesian_nearest_index
#. for step 4, we can use
estimate_mc
(see #658)@oyamad Could you please confirm that you agree?
@Smit-create @HumphreyYang might you be willing to collaborate on this issue?
The text was updated successfully, but these errors were encountered: