Get ROI count#4548
Conversation
dominikl
force-pushed
the
get_roi_count
branch
2 times, most recently
from
af28406 to
d9653dd
Compare
|
Should I open it against regions branch? That was my initial intention, but then I thought, it might be good to get it into develop, that's why I openend the PR against develop for now. |
|
Just be aware that if these are classes you're also changing on regions then there may well be awkward conflicts to resolve: not a showstopper, but takes extra effort. My policy lately has largely been to do stuff on develop only for bugfixes or for classes I don't expect us to touch on regions before |
| * The {@link SecurityContext} | ||
| * @param imageId | ||
| * The image Id | ||
| * @return See above |
There was a problem hiding this comment.
should probably mention it returns -1 in case of error
|
You were right, it conflicts with my regions PR. I'll close this PR and open it against regions. Can be back ported to develop later anyway, if needed. |
|
Moved to regions branch, see #4550 |
3 participants
At the moment Insight loads all ROIs and counts them on the client side for showing the number of ROIs in the right hand side meta data panel. This PR adds a method for directly getting the number of ROIs via an HQL query.
Test: Check some images with ROIs, make sure the ROI count (right hand side metadata panel) is correct.