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

[REF TEST] Refactor data handling #1045

Merged

Conversation

NicolasGensollen
Copy link
Member

@NicolasGensollen NicolasGensollen commented Jan 8, 2024

This PR proposes to refactor the utilities in the clinica.iotools.utils module.

@NicolasGensollen NicolasGensollen self-assigned this Jan 8, 2024
@NicolasGensollen NicolasGensollen added the enhancement New feature or request label Jan 8, 2024
@NicolasGensollen NicolasGensollen added this to the v0.9.0 milestone Apr 26, 2024
@NicolasGensollen NicolasGensollen force-pushed the refactor-data-handling branch 2 times, most recently from 4e7c480 to 35297a9 Compare June 18, 2024 07:09
@NicolasGensollen NicolasGensollen force-pushed the refactor-data-handling branch from 35297a9 to 8928e63 Compare July 3, 2024 13:27
@NicolasGensollen NicolasGensollen force-pushed the refactor-data-handling branch from 8928e63 to a62c6f8 Compare July 22, 2024 14:00
@NicolasGensollen NicolasGensollen marked this pull request as ready for review July 23, 2024 10:04
@NicolasGensollen NicolasGensollen force-pushed the refactor-data-handling branch from 4ef0bd7 to 30f0be7 Compare July 24, 2024 13:24
@NicolasGensollen NicolasGensollen merged commit 7a6a7ca into aramis-lab:dev Jul 25, 2024
14 of 15 checks passed
@NicolasGensollen NicolasGensollen deleted the refactor-data-handling branch July 25, 2024 06:49
AliceJoubert pushed a commit to AliceJoubert/clinica that referenced this pull request Aug 22, 2024
* Add error handling to caller of center_nifti_origin

* start refactoring

* continue refactoring and add tests

* add test

* refactor _create_merge_file_from_bids (BROKEN...)

* better names and more tests

* improve tests and code

* refactor further

* remove duplicate and broken tests

* refactor pipeline handling

* port changes from PR 1052

* port changes from PR 1055

* port changes from PR 1062

* fixes

* fix broken test

* some fixes and enhancements

* fix broken exception
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant