Skip to content

Remove outdated v3 code#2355

Merged
VeckoTheGecko merged 6 commits intoParcels-code:v4-devfrom
VeckoTheGecko:remove-v3-code
Nov 3, 2025
Merged

Remove outdated v3 code#2355
VeckoTheGecko merged 6 commits intoParcels-code:v4-devfrom
VeckoTheGecko:remove-v3-code

Conversation

@VeckoTheGecko
Copy link
Contributor

Fixes #2255
Changes:

  • Rename interpolation_utils.py to place in _core/utils/interpolation.py
  • Remove timer.py
  • Remove interpolators.py
  • Move some tests, remove unused utils

Tree difference for src/parcels:

--- old-tree.txt	2025-11-03 14:00:28
+++ new-tree.txt	2025-11-03 14:00:37
@@ -16,6 +16,7 @@
 │   ├── statuscodes.py
 │   ├── utils
 │   │   ├── __init__.py
+│   │   ├── interpolation.py
 │   │   ├── string.py
 │   │   ├── time.py
 │   │   └── unstructured.py
@@ -34,7 +35,6 @@
 │   │   └── generic.py
 │   └── utils.py
 ├── _decorators.py
-├── _interpolation.py
 ├── _logger.py
 ├── _python.py
 ├── _reprs.py
@@ -63,11 +63,6 @@
 │   ├── advection.py
 │   ├── advectiondiffusion.py
 │   └── interaction.py
-├── tools
-└── utils
-    ├── __init__.py
-    ├── _helpers.py
-    ├── interpolation_utils.py
-    └── timer.py
+└── tools
 
-13 directories, 58 files
+12 directories, 54 files

@VeckoTheGecko
Copy link
Contributor Author

This is probably easiest to review commit by commit

Copy link
Member

@erikvansebille erikvansebille left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@VeckoTheGecko VeckoTheGecko enabled auto-merge (squash) November 3, 2025 14:45
@VeckoTheGecko VeckoTheGecko enabled auto-merge (squash) November 3, 2025 14:45
auto-merge was automatically disabled November 3, 2025 14:51

Pull request was closed

@github-project-automation github-project-automation bot moved this from Backlog to Done in Parcels development Nov 3, 2025
@VeckoTheGecko VeckoTheGecko reopened this Nov 3, 2025
@github-project-automation github-project-automation bot moved this from Done to Backlog in Parcels development Nov 3, 2025
@VeckoTheGecko VeckoTheGecko merged commit 420d91e into Parcels-code:v4-dev Nov 3, 2025
9 of 11 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Parcels development Nov 3, 2025
@VeckoTheGecko VeckoTheGecko deleted the remove-v3-code branch January 20, 2026 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Remove outdated v3 code

2 participants