Skip to content

Miscellaneous utility improvements #1359

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.

Sign up for GitHub

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

Jump to bottom
Open
MaximeBICMTL wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Miscellaneous utility improvements #1359

MaximeBICMTL wants to merge 1 commit into from

Conversation

@MaximeBICMTL
Copy link
Contributor

@MaximeBICMTL MaximeBICMTL commented Jan 8, 2026 •
edited
Loading

Description

Small improvements to the utilities module extracted from #1335.

Details

  • Move path-related functions (= no file system operation) out of lib.util.fs into a new lib.util.path module.
  • Re-order parameters of iterator/collection functions such that the iterator/collection is always the first parameter, which is arguably more intuitive (can expand on this if needed).
  • Note that all the changed/moved functions in this PR are actually not used in the codebase yet(they were extracted from a previous version of MEG BIDS importer #1335), so there is no need to change the call sites . However, I'd like to extract other things from MEG BIDS importer #1335 that build upon those, hence this PR first.

@github-actions github-actions bot added the Language: Python Issue or PR related to the Python codebase label Jan 8, 2026
@MaximeBICMTL MaximeBICMTL added Complexity: Simple Issue or PR that should be simple to implement, review, or test Package: Utilities PR or issue related to the LORIS-agnostic utilities Language: Python Issue or PR related to the Python codebase and removed Language: Python Issue or PR related to the Python codebase labels Jan 8, 2026
@MaximeBICMTL MaximeBICMTL changed the title Improve utilities Miscellaneous utility improvements Jan 8, 2026
@MaximeBICMTL MaximeBICMTL mentioned this pull request Jan 9, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jeffersoncasimir jeffersoncasimir Awaiting requested review from jeffersoncasimir

Assignees

No one assigned

Labels

Complexity: Simple Issue or PR that should be simple to implement, review, or test Language: Python Issue or PR related to the Python codebase Package: Utilities PR or issue related to the LORIS-agnostic utilities

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MaximeBICMTL