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

Update zooom function to support refactored chartSeries function #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

erichung0404
Copy link

Since S4 method used by chartSeries is deprecated, xdata and xsubset should
now be obtained from .plotxtsEnv environment.
plot object needs to be returned by calling return() or being executed at the end
of the code otherwise it will not be printed. To print both the plot object and
"done" message, print() is used. But passing the plot object created by zooom()
to an object is not supported.

data("sample_matrix")
x <- sample_matrix
chartSeries(x)
x.subset <- zooom()
x.subset
# NULL

As S4 method used by chartSeries is deprecated, xdata and xsubset should
be obtained from .plotxtsEnv environment.

plot object is printed if users want to select period more than once
(n>1).
@erichung0404 erichung0404 changed the title Refactor zooom to zoom new chartSeries object Refactor zooom function to manually select subset period Aug 5, 2016
@erichung0404 erichung0404 changed the title Refactor zooom function to manually select subset period Update zooom function to support refactored chartSeries function Aug 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants