Skip to content

Switch to dev xgcm dependency, and add tuna devtool#2550

Merged
VeckoTheGecko merged 5 commits intoParcels-code:mainfrom
VeckoTheGecko:import
Mar 19, 2026
Merged

Switch to dev xgcm dependency, and add tuna devtool#2550
VeckoTheGecko merged 5 commits intoParcels-code:mainfrom
VeckoTheGecko:import

Conversation

@VeckoTheGecko
Copy link
Contributor

Improves the import of parcels by 2.5s for the timebeing until a new version of xgcm is released.

Also adds tuna as a devtool

xref #2546

(only downside of the xgcm unreleased dependency is it might stretch the capacity of asv benchmarking, the support for Rattler (ie. the build system Pixi uses) is limited/buggy)

Using an open PR that has improvements to the import mechanism
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; one small suggestion below

Co-authored-by: Erik van Sebille <e.vansebille@uu.nl>
@VeckoTheGecko VeckoTheGecko enabled auto-merge (squash) March 18, 2026 16:59
@VeckoTheGecko
Copy link
Contributor Author

AH, CI failing because I think I found a bug in Pixi. Investigating.......

@VeckoTheGecko
Copy link
Contributor Author

OK - looks like its due to prefix-dev/pixi#5256 which is being fixed by a change to the lockfile format itself conda/rattler#1987 (it's merged, just waiting for v7 of the lock file to be released - we're on v6 at the moment (which can be seen at the top of the pixi.lock file))

I'm going to see if we can patch CI

@VeckoTheGecko VeckoTheGecko merged commit ff06e47 into Parcels-code:main Mar 19, 2026
11 of 12 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Parcels development Mar 19, 2026
@VeckoTheGecko VeckoTheGecko deleted the import branch March 19, 2026 09:03
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.

2 participants