Versions Compared

Old Version 13

changes.mady.by.user Erik Ziegler

Saved on

compared with

New Version 14

changes.mady.by.user Erik Ziegler

Saved on

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

IssueReasonPlan
imageCache :: memory leak (critical)Large DICOM files with sharedCacheKey will crash the browserMerge immediately
Fix #82: drawInvalidated() should pass trueRequired for requestAnimationFrame support
Display the images that have only VOI LUT Sequence properlyFixes bug with VOI LUT support
  •  Former user (Deleted) will review Evren Ozkan's PR and if it is unacceptable, he will close it. He has investigated this issue and has another fix which may be more appropriate.


Making the canvas, context and its rendering data specific to each enabledElementFixes race condition with renderingMerge immediately
WIP: Prototype of multi-layer compositionDevelopment PRLeave open
Make cornerstone Promise/A+ compliantNot sure we intend to make Cornerstone Promise compliant
  • LoadImage from imageLoaders needs to return deferred
  • Everywhere else can return promises?
  • Should we allow the dev to specify what type of Promises / Promise library Cornerstone should use?
  • Fix imageLoaders to return Deferred
  • Everywhere else can return promises
  • Fix documentation to specify imageLoaders should return Deferred
  • Why do we want to use Deferreds?:
    • So we can cancel the image loading
Rendering loopUses requestAnimationFrame.Review: Not sure it's the best approach
Add possibility to display image staticallyUseful for PreviewsReview / Merge
Rendering using css transform instead of canvas transformDevelopment PRLeave open (or close?)
Added canvasToPixel function for easier use in cornerstoneToolsHelper function which will help us clean up Cornerstone ToolsMerge immediately
WIP: WebGL renderingAdds optional WebGL rendering path. This is stable and has been used in OHIF for months.Merge immediately
Renamed removeImagePromise and return a boolean.
Not sure this is relevant anymore?

...