cbarbu / R-package-zoom

An R package to zoom in and out and more generally navigate a plot or a group of plots in R.
GNU Lesser General Public License v3.0
5 stars 1 forks source link

Error when scrolling with mouse #7

Open scottkosty opened 5 years ago

scottkosty commented 5 years ago

I can scroll with + but when I try to scroll with my mouse, I get an error. Here's a minimal example:

library(zoom)
plot(1:10)
zm()

After running that, if I scroll with my mouse, I get the following error:

Error in if (mevent == "scrollDown") { : argument is of length zero

Here is some information on my system:

> sessionInfo()
R version 3.5.2 (2018-12-20)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 19.04

Matrix products: default
BLAS: /usr/lib/x86_64-linux-gnu/atlas/libblas.so.3.10.3
LAPACK: /usr/lib/x86_64-linux-gnu/atlas/liblapack.so.3.10.3

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8     LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8      
 [8] LC_NAME=C                  LC_ADDRESS=C               LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C       

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base     

other attached packages:
[1] colorout_1.2-0

loaded via a namespace (and not attached):
[1] compiler_3.5.2 parallel_3.5.2
> 

Can you reproduce?

By the way, thank you for your work on this nice package. It made it easy to debug a specific problem I was having.

JohannesTimm commented 5 years ago

I have the same issue. With a feshly installed version of zoom from CRan. After the bug occurs the window does not respond any more to new inputs.

           _                           

platform x86_64-pc-linux-gnu
arch x86_64
os linux-gnu
system x86_64, linux-gnu
status
major 3
minor 4.4
year 2018
month 03
day 15
svn rev 74408
language R
version.string R version 3.4.4 (2018-03-15) nickname Someone to Lean On

cbarbu commented 4 years ago

Hi, I can definitely reproduce. Unfortunately, R seems to have dropped the support of scroll events: an event is returned but without distinguishing scroll up and scroll down. I asked a stackoverflow question, but for now, the last version on github is not offering to scroll and should not fail when doing so.

scottkosty commented 4 years ago

@cbarbu thanks for your response and for maintaining this package. Just for archival purposes, I think the commit fe10d9cb16224e3f30f53af097190704c7a36bb0 is related to this issue.

scottkosty commented 4 years ago

I'm looking into the regression (in R) and along the way came across #8.

scottkosty commented 4 years ago

Here is the upstream bug report: https://bugs.r-project.org/bugzilla/show_bug.cgi?id=17684 Since I don't think this GH issue is due to a bug in the zoom package, feel free to close it.

cbarbu commented 4 years ago

Thanks for doing the bug report.

On Sun, Jan 5, 2020 at 5:41 AM Scott Kostyshak notifications@github.com wrote:

Here is the upstream bug report: https://bugs.r-project.org/bugzilla/show_bug.cgi?id=17684 Since I don't think this GH issue is due to a bug in the zoom package, feel free to close it.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cbarbu/R-package-zoom/issues/7?email_source=notifications&email_token=AAEBIBFLFDBHFXLI3VQUV7DQ4FQJJA5CNFSM4HSEETT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIDJTJA#issuecomment-570857892, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEBIBAXZOZFSUACMLFPZW3Q4FQJJANCNFSM4HSEETTQ .

--


Corentin M. Barbu-Covantes (484) 843-1580 http://scholar.google.com/citations?hl=en&user=sxDMRdQAAAAJ "For what does it profit a man to gain the whole world, and forfeit his soul?" Mark 8:36