search

zooniverse / Zooniverse

27 stars 13 forks source link

Changes to subject queue #33

Closed brian-c closed 10 years ago

brian-c commented 10 years ago

@parrish, this look good for #31?

parrish commented 10 years ago

Looks good. My only suggestion would be to store seen subject id's in an Object to give constant-time lookups. Not sure if there's even a noticeable performance/memory hit with an array of less than 1,000 elements though.

Thanks!

brian-c commented 10 years ago

I'm betting it won't be noticeable in an array of 1,000, and this lets us keep the list a manageable size.