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

Convert CLIs from C++ CLIs to Python Scripted modules + add BackScatterCoefficient module #82

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

jcfr
Copy link
Contributor

@jcfr jcfr commented Aug 16, 2023

This pull request depends on:

Module Updates

Modules Before/After
ApplyButterworthHighpass1D No change
BackScatterCoefficient ✨ New scripted 🐍 module
BModeFromRF ✨ New scripted 🐍 module (previously called GenerateBModeFromRF)
GenerateBModeFromRF ❌ Removed CLI module
GenerateDisplacementFromFrames No change
GenerateDisplacementFromTimeSeries No change
ScanConvertCurvilinearArray ✨ Converted from CLI to Scripted 🐍 module
ScanConvertPhasedArray3D ✨ Converted from CLI to Scripted 🐍 module
ScanConvertSliceSeries ✨ Converted from CLI to Scripted 🐍 module

Also added support modules:

  • ITKUltrasoundCommon
  • ScanConvertCommon

dzenanz and others added 21 commits March 6, 2023 12:10
This module houses common functionality for a few scan conversion modules.
Start ScanConvertPhasedArray3D by copying ScanConvertCurvilinearArray
Split lateral separation parameter into azimuth and elevation
Update input image information
Update test
Require itk-ultrasound 0.6.2 for PA3DSCI wrapping
Do not reference non-existent UI resource for ITKUltrasoundCommon
Add ScanConvertCommon to the extension's list of modules
Rename ScanConvertCommon.png
…xt manager

Start with Slicer/Slicer@19d2cbe4c (ENH: Add context managers for
message and error display to slicer.util) introduced in Slicer 5.0,
the slicer.util.WaitCursor context manager implements the corresponding
functionality.
This commit removes the following functions originally introduced
in 1fdd2f8 (ENH: BMode filter works):
* get_rotation_and_spacing_from_affine
* get_itk_metadata_from_ras_affine
* get_ras_affine_from_itk
@dzenanz
Copy link
Member

dzenanz commented Aug 16, 2023

Thanks for your work on this. This is no longer at the top of my priority list, so it will take a while for me to return to this work and finish this refactoring. Let's keep this PR open until then.

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