-
Notifications
You must be signed in to change notification settings - Fork 42
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
oce.plot.ts() fails with "need finite 'ylim' values" even though plot() works #2277
Comments
I'll look into this tomorrow. That bit of the code is a bit on the tricky side. But I like knowing about this issue, because it has not come up in my usual workflow (where there are normally more finite values than NA ones; here there are 6 NA values per non-NA value. Glider data are fun :-) |
It now checks for the condition. If Tomorrow I'll look into seeing why simplification does this. There may be a way to retain some data. But I need to think through what makes most sense, in the "R way of thinking", i.e. what users would expect.
|
The problem is that the data just happen to be sparser than I had thought could occur. The way simplification works for If there is only one value there, then the min will equal the max and so the vertical line will have zero extent. But maybe that's OK. At least it seems OK to me right now. I need to mess with some sample datasets and also to scratch my head for a while. Maybe sometime if @richardsc and I are in a room together, we can chat about this, since simplification was very much a jointly -developed idea between the two of us. |
The dataset is from a glider and so creating a regexp is a bit tricky. I plan to instrument the code and to see where the problem lies. Then I may be able to see how to make a regexp, and I'll insert that (possibly with a better titles).
The text was updated successfully, but these errors were encountered: