-
Notifications
You must be signed in to change notification settings - Fork 211
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
Bulk loading (with STR) #45
Comments
Thanks I'll have a look. If the data is static you can use serialisation by On Thu, 14 Apr 2016 01:27 x2b notifications@github.com wrote:
|
hi @davidmoten Can you review my implementation of this feature? |
Do you observe a speed improvement relative to the RTree creation? Update: sorry forget my comment I have just seen that the comparison was made in the related fork: Thanks! |
Hi, actually I am still working on the performance evaluation. Sorry for the delay. I have tried to add more test cases in this benchmark class in this branch, however it seems to mess up the codebase and it is very tedious to make some changes to the rtree types or datasets. So I opened a new project according to the guide of jmh, and trying to make the code more clean with help of the inheritance feature. I am no familiar with jmh, so I would greatly appreciate if you guys have any suggestions on this benchmark project. BTW, maybe this issue is safe to close and leave me some more days to open a new PR to update the README docs after I have mature benchmarking results? |
ok closing, thanks for your efforts @ambling |
@ambling: Thank you for setting up this benchmarking project. |
I have a rather large set of entries (>= 300k) and I would like to somewhat improve the performance of the tree. The data is available beforehand so I think that bulk loading might help, in particular the "Sort-Tile-Recursive" strategy. I think this could be implemented fairly easily.
The text was updated successfully, but these errors were encountered: