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

Clustering items #53

Open
next2heaven opened this issue Dec 4, 2012 · 1 comment
Open

Clustering items #53

next2heaven opened this issue Dec 4, 2012 · 1 comment

Comments

@next2heaven
Copy link

Been testing the new clustering. Looking good! The only 2 issues I see is that depending on event heights it's doesn't bucket properly. I think the calculation being used is expecting each row to be a certain height so it ends up hiding some items when you have variable hight items on the timeline.

The other thing is having an option to click the bucket and the timeline will zoom till all items in the bucket can safely be shown. I know that's on the list.

This is such a big item. Glad to see it worked on. Keep up the great work!

@josdejong
Copy link
Contributor

Thanks. This first solution does indeed not take into account the size of the items, but only their timestamp, and does not take into account the stacked height at all. This makes clustering easy and relatively fast: you can filter and cluster before rendering items.

It is an interesting problem: suppose you have 1m items. If you do take into account their actual size, you will have to render them all once in order to determine their size. Only after that you can determine what the stacked height will be, and and cluster them accordingly. But that rendering is exactly the most expensive part.
A solution can be to specify a default size for the items, which more or less corresponds to the actual size, and use this default size as an approximation as long as the item hasn't been rendered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants