Fixed pre-commit | Added one unit test

[spirosmaggioros]

• Removed unused imports DLMUSE and DLICV from SegmentImage.py.

• Corrected a typo in the docstring of the merge_output_data function in utils.py.

• Added type hint Any to the progress_bar parameter in the run_pipeline function in dlmuse_pipeline.py.

• Added a new test function testing_get_bids_prefix in test_utils.py to validate the get_bids_prefix function.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.42%. Comparing base (976fa36) to head (58dda13).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
+ Coverage   71.03%   72.42%   +1.39%     
==========================================
  Files           3        3              
  Lines         290      301      +11     
==========================================
+ Hits          206      218      +12     
+ Misses         84       83       -1     

Files with missing lines	Coverage Δ
NiChart_DLMUSE/__init__.py	100.00% <100.00%> (ø)
NiChart_DLMUSE/utils.py	57.36% <100.00%> (+0.52%)	⬆️
tests/test_utils.py	98.18% <100.00%> (+0.20%)	⬆️

[AlexanderGetka-cbica]

Regarding the imports, we should rewrite the run_pipeline, run_dlicv, run_dlmuse methods in this package to directly call the DLMUSE and DLICV packages rather than going through the CLI. This will be useful in the future for more in-depth progress reports, error reporting/handling, and parallelization at the level of these modules

[spirosmaggioros]

Agree. I didn't change kyle's logic so i kept it as is.
Will do this on another branch tomorrow!

[spirosmaggioros]

@AlexanderGetka-cbica I will wait for my PR's at DLMUSE and DLICV to be reviewed/merged before merging this one as i will add the direct api calls for DLMUSE and DLICV.