Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upFast aoi queries #2
Conversation
# Conflicts: # DESCRIPTION # README.md
makes the extraction of rectangle/circles faster
|
Thanks, but
|
| @@ -23,7 +23,7 @@ | |||
| implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. | |||
| CHANGE HISTORY: | |||
| 3 February 2017 -- by Florian de Boissieu -- L1798,1813 added parse_str because of strange problem with overloaded functions | |||
Jean-Romain
Feb 3, 2017
•
Owner
I know that there is a strange problem with overloaded functions. I'm ok with that. But why you needed a new function. There is no function to parse CHAR?
I know that there is a strange problem with overloaded functions. I'm ok with that. But why you needed a new function. There is no function to parse CHAR?
|
|
||
| npoints = 0; | ||
| while (lasreader->read_point()) | ||
| { | ||
| if(!lasfilter.filter(&lasreader->point)) | ||
| // if(!lasfilter.filter(&lasreader->point)) |
Jean-Romain
Feb 3, 2017
Owner
It means the reader skip automatically the points based on the filter which is now into the reader. Right?
It means the reader skip automatically the points based on the filter which is now into the reader. Right?
| @@ -142,8 +144,8 @@ List lasdatareader(CharacterVector file, bool Intensity, bool ReturnNumber, bool | |||
| unsigned long int i = 0; | |||
| while (lasreader->read_point()) | |||
| { | |||
| if(stream && lasfilter.filter(&lasreader->point)) | |||
| continue; | |||
| // if(stream && lasfilter.filter(&lasreader->point)) | |||
Jean-Romain
Feb 3, 2017
Owner
Btw same comment than the previous one.
Btw same comment than the previous one.
| LASreader* lasreader = lasreadopener.open(); | ||
| LASfilter lasfilter; | ||
| // LASfilter lasfilter; |
Jean-Romain
Feb 3, 2017
Owner
It means the filter is integrated into the reader. Right?
It means the filter is integrated into the reader. Right?
I'm guilty. master and devel were not up to date. Moreover Dave pushed in master instead of devel. So it was messy. master and devel are now even. Update your fork. Edit: you're guilty too. You worked in master an pushed in devel. |
changed lasfilter queries with lasreader fast AOI queries Former-commit-id: b4b3b72
Changed lasfilter queries for lasreader fast AOI queries, which makes the extraction of rectangle/circle faster with indexed las/laz files (see link).