Adding basic Processing version of the JS QuadTree #16
Conversation
| return false; | ||
| } | ||
|
|
||
| ArrayList<Point> query(Rectangle range, ArrayList<Point> found) { |
There was a problem hiding this comment.
Should this be List rather than ArrayList?
There was a problem hiding this comment.
ArrayList is good b/c it's what I use as standard catch-all flexible array in my tutorials and examples!
|
I wonder if this might make more sense as a separate repo? Rather than have both in this repo? |
shiffman
changed the title
|
@shiffman I think making it part of a separate repo would work better, so that this repo keeps focus on it being JS library. We can then port examples to java as well to match the examples here. On the other hand, you did mention in a previous livestream that you were interested in creating some sort of processing library on eclipse CC. Can both things happen concurrently? |
|
Hi! I'm revisiting the QuadTree stuff as I'm teaching it in my class next week. @Carla-de-Beer would you mind posting this code in your own repo and then adding a link to it from the README here? |
|
Of course, yes. You are correct, it makes more sense to keep the P5,js repository clean. I added a new pull request (Update README.md #35) to allow for this. Apologies if I caused any inconvenience. |
*version