Added statistics calculator for Annotation Tools #723
Conversation
✅ Deploy Preview for cornerstone-3d-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
@sedghi Hey, I've been working on annotation tools and I added the feature to custom the text shown by the tool. So we can add custom text and also custom statistics calculator by the tool configuration. So now we can set our custom text and calculator when adding the tool into the toolgroup with it's configuration or also by I've created a new example aswell (stackAnnotationToolsSpecificConfiguration) |
packages/tools/src/utilities/math/basic/BasicStatsCalculator.ts
Outdated
Show resolved
Hide resolved
packages/tools/src/utilities/math/basic/BasicStatsCalculator.ts
Outdated
Show resolved
Hide resolved
packages/tools/src/utilities/math/basic/BasicStatsCalculator.ts
Outdated
Show resolved
Hide resolved
packages/tools/src/utilities/math/basic/BasicStatsCalculator.ts
Outdated
Show resolved
Hide resolved
| addTool: { | ||
| ( | ||
| toolName: string, | ||
| toolConfiguration?: Record<any, any> & { statsCalculator?: Calculator } |
There was a problem hiding this comment.
This should be ToolConfiguration
There was a problem hiding this comment.
Oh true , I forgot about it. I'll update it now
| imageData, | ||
| (pointLPS, pointIJK) => pointInEllipse(ellipseObj, pointLPS), | ||
| stdCalculator, | ||
| this.configuration.statsCalculator.run, |
There was a problem hiding this comment.
ok, I guess my bad, .run reads too werid, let's have your callback back. Sorry about that
There was a problem hiding this comment.
Sure, no worries
Context
Refactored the stats calculation code for Annotation Tools and added a way to create and initialize new calculator with custom statistics.
Changes
Testing
stackAnnotationTools example.
Checklist
PR
semantic-release format and guidelines.
Code
[] My code has been well-documented (function documentation, inline comments,
etc.)
[] I have run the
yarn build:update-apito update the API documentation, and havecommitted the changes to this PR. (Read more here https://www.cornerstonejs.org/docs/contribute/update-api)
Public Documentation Updates
additions or removals.
Tested Environment