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

fix(cpu): could not render if switched to cpu in the middle #615

Merged
merged 3 commits into from
May 18, 2023

Conversation

sedghi
Copy link
Member

@sedghi sedghi commented May 17, 2023

Context

CPU rendering if it is switched after the initial setup would fail to use renderToCanvas.

Changes & Results

  • Fixed the CPU rendering to use the correct rendering function
  • refactor some types
  • make pallet color to use metadata to fetch if needed

Testing

  1. go to https://v3-demo.ohif.org/localbasic
  2. open console and say cornerstone.setUseCPURendering(true)
  3. drag and drop a dicom
  4. thumbnail would not load

Checklist

PR

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

Code

Public Documentation Updates

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

Tested Environment

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

@netlify
Copy link

netlify bot commented May 17, 2023

Deploy Preview for cornerstone-3d-docs canceled.

Name Link
🔨 Latest commit 101bc1f
🔍 Latest deploy log https://app.netlify.com/sites/cornerstone-3d-docs/deploys/6466654cf3eb6400088f0cb1

@sedghi sedghi requested a review from jbocce May 17, 2023 19:36
Copy link
Contributor

@jbocce jbocce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks for the testing notes. It worked nicely.

@sedghi sedghi merged commit 6b1d588 into main May 18, 2023
10 checks passed
iamiota added a commit to iamiota/cornerstone3D-beta that referenced this pull request May 26, 2023
* 'main' of github.com:iamiota/cornerstone3D-beta: (59 commits)
  chore(release): publish [skip ci]
  fix(event): Interactions between double click and multi mouse button (cornerstonejs#616)
  chore(release): publish [skip ci]
  fix(windowLevel): should not double scale for window level (cornerstonejs#619)
  chore(release): publish [skip ci]
  fix(expose): api default mouse primary (cornerstonejs#622)
  chore(release): publish [skip ci]
  docs(typo): in imageLoader.md (cornerstonejs#629)
  chore(release): publish [skip ci]
  fix(husky): husky config (cornerstonejs#627)
  chore(release): publish [skip ci]
  feat(api): cornerstone3D (cornerstonejs#626)
  chore(release): publish [skip ci]
  feat(api)!: cornerstone3D version 1.0 (cornerstonejs#625)
  chore(release): publish [skip ci]
  feat(version1)!: publish version 1.0 (cornerstonejs#624)
  chore(release): publish [skip ci]
  fix(versioning): sync all versions (cornerstonejs#623)
  chore(release): publish [skip ci]
  fix(cpu): could not render if switched to cpu in the middle (cornerstonejs#615)
  ...
@sedghi sedghi deleted the fix/some-more-bugs branch August 2, 2023 20:32
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.

None yet

2 participants