Skip to content

Conversation

@MaximeBICMTL
Copy link
Contributor

@MaximeBICMTL MaximeBICMTL commented Jun 18, 2025

Description

This PR deleted the deprecated and unused code in LORIS-MRI 27. This allows to reduce the amount of code to maintain and the depreciation warnings in CI. All removed code have @deprecated annotations that point to newer alternatives.

Changelog

  • Remove module lib.database_lib.mri_scanner.
  • Remove module lib.database_lib.mri_upload_db.
  • Remove module lib.database_lib.notification.
  • Remove module lib.database_lib.project_cohort_rel.
  • Remove module lib.database_lib.session_db.
  • Remove module lib.database_lib.site.
  • Remove module lib.database_lib.tarchive.
  • Remove module lib.database_lib.tarchive_series.
  • Remove module lib.database_lib.visit_windows.
  • Remove module lib.dicom_archive.
  • Remove module lib.imaging_io.
  • Remove module lib.imaging_upload.
  • Remove module lib.log.
  • Remove method lib.imaging.get_scanner_id.
  • Remove non-BIDS-related methods in lib.session.
  • Remove hashing methods in lib.utilities.

@MaximeBICMTL MaximeBICMTL requested a review from cmadjar July 1, 2025 07:02
@MaximeBICMTL MaximeBICMTL marked this pull request as ready for review July 1, 2025 07:02
@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 20 days.

@github-actions github-actions bot added the Stale label Sep 30, 2025
@MaximeBICMTL MaximeBICMTL removed the Stale label Oct 6, 2025
@MaximeBICMTL MaximeBICMTL force-pushed the delete_unused_deprecated_code branch from a65ca6b to c75b51c Compare November 6, 2025 15:22
@MaximeBICMTL
Copy link
Contributor Author

Meh too lazy to fix the broken test for now. That test will be easily fixed once the BIDS importer is merged.

@MaximeBICMTL MaximeBICMTL added the Caveat for Existing Projects Issue or PR that introduces or may introduce breaking changes for existing projects label Nov 6, 2025
@MaximeBICMTL MaximeBICMTL added the Language: Python Issue or PR related to the Python codebase label Nov 17, 2025
@MaximeBICMTL MaximeBICMTL force-pushed the delete_unused_deprecated_code branch from c75b51c to f4bca8b Compare January 9, 2026 04:48
@MaximeBICMTL MaximeBICMTL force-pushed the delete_unused_deprecated_code branch from f4bca8b to 1b1a23e Compare January 9, 2026 04:50
@MaximeBICMTL MaximeBICMTL marked this pull request as draft January 15, 2026 15:52
@MaximeBICMTL MaximeBICMTL removed the request for review from cmadjar January 15, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Caveat for Existing Projects Issue or PR that introduces or may introduce breaking changes for existing projects Cleanup Language: Python Issue or PR related to the Python codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant