Skip to content

Conversation

@Sofien-Sellami
Copy link

Context

Changes & Results

Testing

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:

salimkanoun pushed a commit that referenced this pull request Dec 14, 2024
* feat: dynamic threshold sphere

* Add dynamic sphere

* fix: Performance issues on threshold sphere

* performance

* feat: Create segmentation voxel manager in initial create of volume

* Faster fill for fixing features

* performance: Fix the performance of island removal

* fix restore of mixed islands

* fix external island removal on non-acquisition

* Performance and planar filling improvements

* Fix flood fill not being planar

* Remove invalid points in shape

* feat: Add a labelmap statistics calculator (#2)

* fix: Change to names for statistics so that other implementations can be added

* Fix build issues

* feat: Add working statistics calcs

* fix: Add stats to the labelmap calculator

* Compute labelmap statistics and lesion glycolysis

* PR comments

* PR comments

* PR - comments added

* PR review comments

* PR fixes - mostly cleanup

* Improvements to threshold out of plane

* Missed a fix on the sphere change

* fix

* fix

* fix

* wip

* api

* fix: RLE rendering for segmentations

* fix: Undefined error on reject preview

* wip

* wip

* refactoring

* fix

* Enhance VoxelManager and RenderingEngine:
- Updated `getMinMax` method in `VoxelManager` to handle array inputs for min/max calculations.
- Modified `setDefaultVolumeVOI` to correctly compute min/max values from voxel data.
- Cleaned up and restructured tests in `volumeViewport_gpu_render_test.js`, restoring necessary imports and ensuring proper rendering tests for various volume types.

* update

* bring back tests

* update

---------
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