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

compare.gmwm #199

Open
philippcla opened this issue Sep 6, 2016 · 8 comments
Open

compare.gmwm #199

philippcla opened this issue Sep 6, 2016 · 8 comments

Comments

@philippcla
Copy link

Hello,
I am using this awesome function:
compare.gmwm(estimated_model_one, estimated_model_two, estimated_model_three, split = FALSE)

Unfortunately this line of code gives me an error, as I can only provide 2 models at a time. Would it be possible to have an unlimited number of models as input?

Thanks
Philipp

@Wenchao-Yang
Copy link
Member

The new feature will be in the github very soon. I will let you know when it's ready.

@philippcla
Copy link
Author

perfect
thx

@coatless
Copy link
Contributor

coatless commented Sep 6, 2016

The output will look slightly different than the present compare.gmwm() version.

@coatless
Copy link
Contributor

@Wenchao-Yang : I need the feature to be PR'd before next week. The package needs to be updated on CRAN with the new developments.

@coatless
Copy link
Contributor

@philippcla:

Check out:

compare_models(estimated_model_one, estimated_model_two, estimated_model_three)

in the latest development build.

@philippcla philippcla reopened this Feb 8, 2017
@philippcla
Copy link
Author

philippcla commented Feb 8, 2017

Hello,
The option "split = FALSE" does nomore exist in the "compare_models"
How do I plot several models on the same plot?
It is related to the issue #200, where I wanted to plot several models on one plot withouth the huge legend

thx

@coatless
Copy link
Contributor

coatless commented Feb 9, 2017

We actually merged this feature into compare_models() slightly. The lower diagonal of compare_models() mimics the split = FALSE view of two models. We found that anything more than two models became too noisy and was very problematic to interpret especially if the scales between models differed. Furthermore, the key aspect of the comparison was how the implied WV differed. This difference is captured in the upper triangular section of the matrix. Meanwhile, on the diagonal, we have the classic GMWM model plot.

As a result, I'm hesitant to resurrect the old plot. But, let me know.

library('gmwm')
#> Loading required package: ggplot2
set.seed(8836)
x1 = gen_gts(1000, AR1(phi = .1, sigma2 = 1) + AR1(phi = 0.95, sigma2 = .1))
x2 = gen_gts(2000, AR1(phi = .1, sigma2 = 1) + AR1(phi = 0.95, sigma2 = .1))

GMWM1 = gmwm(AR1(), data = x1)
GMWM2 = gmwm(2*AR1(), data = x2)

compare_models(GMWM1, GMWM2, show.theo.wv = T, transparence = 0.2, 
               facet.label = c('model1', 'model2'))

@philippcla
Copy link
Author

Hello
I see your point for the "over-plotting" and then there is too much info on the graph, ok. But this should be left to the user to decide if he want's to keep an unreadable plot.
Could we not have an extra bool in the function, so that I could choose the upper right or lower left figure to be plotted alone.
The lower left plot is nice to show the differences but the CI are a must. So there is less need for the upper right plot.
Could we have a bool in the function allowing to plot just the lower left plot? Or is there another way to just have the lower left plot for the moment?

Cheers

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

No branches or pull requests

3 participants