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

feat(SM): remove SM measurements from measurement panel #4022

Merged
merged 14 commits into from
Apr 10, 2024

Conversation

sedghi
Copy link
Member

@sedghi sedghi commented Apr 10, 2024

will address #3950

Context

Changes & Results

Testing

Checklist

PR

  • [] My Pull Request title is descriptive, accurate and follows the
    semantic-release format and guidelines.

Code

  • [] My code has been well-documented (function documentation, inline comments,
    etc.)

Public Documentation Updates

  • [] The documentation page has been updated as necessary for any public API
    additions or removals.

Tested Environment

  • [] OS:
  • [] Node version:
  • [] Browser:

Copy link

netlify bot commented Apr 10, 2024

Deploy Preview for ohif-platform-docs ready!

Name Link
🔨 Latest commit f21f359
🔍 Latest deploy log https://app.netlify.com/sites/ohif-platform-docs/deploys/6616cd6cc0f0210008c71b21
😎 Deploy Preview https://deploy-preview-4022--ohif-platform-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Apr 10, 2024

Deploy Preview for ohif-dev ready!

Name Link
🔨 Latest commit 0b13044
🔍 Latest deploy log https://app.netlify.com/sites/ohif-dev/deploys/6616b2405dbbd100086f5fc6
😎 Deploy Preview https://deploy-preview-4022--ohif-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@sedghi sedghi changed the title feat/remove sm measurements feat(SM): remove measurements from measurement panel Apr 10, 2024
@sedghi sedghi changed the title feat(SM): remove measurements from measurement panel feat(SM): remove SM measurements from measurement panel Apr 10, 2024
Copy link

netlify bot commented Apr 10, 2024

Deploy Preview for ohif-dev ready!

Name Link
🔨 Latest commit f21f359
🔍 Latest deploy log https://app.netlify.com/sites/ohif-dev/deploys/6616cd6ce3c35a0008e28278
😎 Deploy Preview https://deploy-preview-4022--ohif-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

codecov bot commented Apr 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.37%. Comparing base (8a335bd) to head (957bd13).
Report is 309 commits behind head on master.

❗ Current head 957bd13 differs from pull request most recent head f21f359. Consider uploading reports for the commit f21f359 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4022      +/-   ##
==========================================
- Coverage   46.23%   44.37%   -1.87%     
==========================================
  Files          78       80       +2     
  Lines        1276     1334      +58     
  Branches      312      327      +15     
==========================================
+ Hits          590      592       +2     
- Misses        548      589      +41     
- Partials      138      153      +15     

see 11 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77693dd...f21f359. Read the comment docs.

Copy link

cypress bot commented Apr 10, 2024

2 failed tests on run #3805 ↗︎

2 41 0 0 Flakiness 0

Details:

update cs3d
Project: Viewers Commit: f21f359b5b
Status: Failed Duration: 05:55 💡
Started: Apr 10, 2024 5:51 PM Ended: Apr 10, 2024 5:57 PM
Failed  study-list/OHIFStudyList.spec.js • 1 failed test

View Output Video

Test Artifacts
OHIF Study List > Desktop resolution > maintains Description filter upon return from viewer Test Replay Screenshots Video
Failed  customization/HangingProtocol.spec.js • 1 failed test

View Output Video

Test Artifacts
OHIF HP > Should navigate next/previous stage Test Replay Screenshots Video

Review all test suite changes for PR #4022 ↗︎

@sedghi sedghi merged commit df49a65 into master Apr 10, 2024
6 of 7 checks passed
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.

2 participants