Skip to content

Bugfixes before release#700

Merged
m-reuter merged 1 commit into
Deep-MI:devfrom
dkuegler:fix/type-catch
Aug 12, 2025
Merged

Bugfixes before release#700
m-reuter merged 1 commit into
Deep-MI:devfrom
dkuegler:fix/type-catch

Conversation

@dkuegler
Copy link
Copy Markdown
Member

This PR fixes several bugs that came up and were resolved during release testing and further review.

Fix the usage of nibabel.orientations.ornt_transform after clarification. nipy/nibabel#1418

Fix Argument-Type related bugs.

  • "soft LIA" to "soft-LIA"
  • ArgumentTypeError to ValueError
  • vox_size can accept a float in addition to a str
  • fix image_size default value overwriting passed value
  • remove test from Docker
  • reorder vox_size and image_size check in is_conform so their order in output is swapped
  • set LANG to en_US in Dockerfile
  • fix image position when cropping with crop_transform
  • only reset LC_NUMERIC in recon_surf/functions, if LC_NUMERIC is not en_US

Comment thread FastSurferCNN/utils/arg_types.py
@dkuegler dkuegler marked this pull request as ready for review August 12, 2025 14:27
Comment thread recon_surf/functions.sh Outdated
- "soft LIA" to "soft-LIA"
- ArgumentTypeError to ValueError
- vox_size can accept a float in addition to a str
- fix image_size default value overwriting passed value
- remove test from Docker
- reorder vox_size and image_size check in is_conform so their order in output is swapped
- set LANG to en_US in Dockerfile
- fix image position when cropping with crop_transform
- add inv_ornt function in conform
- remove extra debug output

General fixes to FastSurfer
- only reset LC_NUMERIC in recon_surf/functions, if LC_NUMERIC is not en_US
- set LC_NUMERIC in Dockerfile
- add logfile to softlink_or_copy invocation in run_fastsurfer (was missing)
- add a warning to softlink_or_copy if logfile is not set
- some minor formatting fixes
@m-reuter m-reuter merged commit 69a765f into Deep-MI:dev Aug 12, 2025
3 checks passed
@dkuegler dkuegler deleted the fix/type-catch branch August 15, 2025 09:49
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