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

error in LoadLidar() #1

Closed
ptompalski opened this issue Mar 22, 2016 · 6 comments
Closed

error in LoadLidar() #1

ptompalski opened this issue Mar 22, 2016 · 6 comments
Assignees
Labels
Wontfix Not reproducible or not fixable

Comments

@ptompalski
Copy link
Contributor

When loading a pointcloud with LoadLidar() or readLas() following error occurs:

Error in as.data.table.matrix(xi, keep.rownames = keep.rownames) : 
  4 arguments passed to .Internal(nchar) which requires 3

error does not occur when
return(data.table(mm, Intensity, ReturnNumber,NumberOfReturns,Classification,ScanAngle,R,G,B,gpstime))
is changed into
return(data.frame(mm, Intensity, ReturnNumber,NumberOfReturns,Classification,ScanAngle,R,G,B,gpstime))
readLas.r, line 200

@Jean-Romain
Copy link
Collaborator

Hi,

Please check and report your:

  • R version (mine is 3.2.2)
  • data.table package version (mine is 1.9.6)

I found 3 references with this error: here joey711/phyloseq#564 and here http://stackoverflow.com/questions/31142280/unusual-error-in-setkeyv-with-data-table and here http://grokbase.com/t/r/r-help/156m0j2nfm/r-data-table-package-problem. It seems that the problem comes from the R version.

If updating R solves this issue I will change the DESCRIPTION file.

@ptompalski
Copy link
Contributor Author

R version 3.2.2
data.table 1.9.6

After updating R to 3.2.4 problem does not occur.
This is strange and I am not sure what causes that.

@Jean-Romain
Copy link
Collaborator

If the problem comes from R or data.table, it's out of my skills.

Here with the same version than mine (linux) and the same than another user (windows) you have the problem. You updated to a greater version than the one we are using. I can't modify the DESCRIPTION file to impose a greater version than mine because it works on 3.2.2. The problem comes from somewhere deeper than what I can investigate.

@Jean-Romain Jean-Romain added the Bug A bug in the package label Mar 23, 2016
@Jean-Romain Jean-Romain self-assigned this Mar 23, 2016
@ptompalski
Copy link
Contributor Author

I will try this on several other machines and report back.

Edit:
works on R 3.2.1, dt 1.9.6
works on R 3.1.2, dt 1.9.6
works on R 3.1.2, dt 1.9.6

all on windows platforms.

Hopefully this was a single issue on my laptop.

@Jean-Romain
Copy link
Collaborator

Works also on R 3.2.2 on Windows platform on a friend's computer. We made 7 tests, 6 on Windows, 1 on Linux, on 4 versions of R. I can maybe make more tests on Linux on my other computers but if it's not a single issue it will be very very hard to reproduce it and then, to solve it.

Thank you for your tests. I will consider this bug fixed because we cannot reproduce it. It seems to be specific to something else than pure code.

@ptompalski
Copy link
Contributor Author

Agree. If it happens again, then we can investigate it more.

@Jean-Romain Jean-Romain added Wontfix Not reproducible or not fixable and removed Bug A bug in the package labels Feb 5, 2017
Jean-Romain pushed a commit that referenced this issue Nov 12, 2018
Jean-Romain pushed a commit that referenced this issue Dec 30, 2018
Bringing fork up to date

Former-commit-id: 5391c4313f1fc7f6a1de1c0b65c9cfde5d9d0156
Former-commit-id: 2b3b31fe93e6da95bcd38a9db26d916eec5519ac
Former-commit-id: 108f03325c018bcc9954b5c4ba15ce56f9b156eb [formerly f1f4933b0767d1a9ed842c786b23c5e34298699b]
Former-commit-id: 76ffab218f4ade85763cc48170ee8045d2df6a47
Former-commit-id: 6b3d56ff86a126b91fa75238e432c9155ac81fbf
Former-commit-id: f33331982852ad95fb9224a121b161491a6d628e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Wontfix Not reproducible or not fixable
Projects
None yet
Development

No branches or pull requests

2 participants