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

is_A1 FALSE if number too big -> read_excel doesnt work #37

Open
Droelf-source opened this issue Jul 5, 2021 · 0 comments
Open

is_A1 FALSE if number too big -> read_excel doesnt work #37

Droelf-source opened this issue Jul 5, 2021 · 0 comments

Comments

@Droelf-source
Copy link

Droelf-source commented Jul 5, 2021

Hello,

I noticed that my range given in A1 format doesn't work with read_excel if the number of rows is bigger than 100,000. I think the only problem is that the is_A1(x) check fails. Curiously, if I try

cellranger::is_A1("$A$636026")
FALSE

but if I copy the code from the github repository

.cr <- new.env(parent = emptyenv())
.cr$A1_rx <- "^\$?[A-Za-z]{1,3}\$?[0-9]{1,7}$"
is_A1 <- function(x) { grepl(.cr$A1_rx, test)}
is_A1("$AA$636026")
[1] TRUE

it does work. Now if I try to read in excel files using a range with more than 100,000 rows it stops and says it isn't in A1 format.

Also the breakpoint is in fact 100,000

cellranger::is_A1("$A$100000")
[1] FALSE
cellranger::is_A1("$A$99999")
[1] TRUE

For now I can workaround this issue by replacing the function with the copied code from github but I think it is kinda an interesting as I have no idea how it doesn't work as is.

Best Regards

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

No branches or pull requests

1 participant