Skip to content

New Tool - Height Tool - #1508

Open
ArturRod wants to merge 5 commits into
cornerstonejs:masterfrom
ArturRod:Height-Tool
Open

New Tool - Height Tool#1508
ArturRod wants to merge 5 commits into
cornerstonejs:masterfrom
ArturRod:Height-Tool

Conversation

@ArturRod

@ArturRod ArturRod commented Sep 14, 2022

Copy link
Copy Markdown
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    -> It is a new height measurement tool, to be able to compare height differences, for example, a hip.
  • What is the current behavior? (You can also link to an open issue here)
    -> Measure height:
    image
height-tool.mp4
  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    -> Being a new tool, the user can decide to add it to their projects or not.

@codecov

codecov Bot commented Sep 14, 2022

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 31.18280% with 64 lines in your changes missing coverage. Please review.
✅ Project coverage is 23.93%. Comparing base (dbf874f) to head (eab1836).

Files with missing lines Patch % Lines
src/tools/annotation/HeightTool.js 35.80% 41 Missing and 11 partials ⚠️
src/drawing/drawHeight.js 0.00% 11 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1508      +/-   ##
==========================================
+ Coverage   23.86%   23.93%   +0.06%     
==========================================
  Files         287      289       +2     
  Lines       10141    10234      +93     
  Branches     2082     2102      +20     
==========================================
+ Hits         2420     2449      +29     
- Misses       6575     6627      +52     
- Partials     1146     1158      +12     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Successfully merging this pull request may close these issues.

1 participant