-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add HDBSCAN #783
Add HDBSCAN #783
Conversation
Kokkos::Profiling::ProfilingSection profile_mst("ArborX::HDBSCAN::mst"); | ||
profile_mst.start(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are the regions "ArborX::HDBSCAN"
with the nested "ArborX::MST::MST"
and "ArborX::Dendrogram::Dendrogram"
not sufficient?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are profiling sections that we'll use to print out the timers. When using Kokkos tooling like space-time-stack
, those would have been sufficient.
8c2e7d0
to
b303604
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will try to refactor to replace profiling sections in a follow up
Contains first two parts, but does not contain flat clustering right now.