Skip to content

Better ASV Pixi integration#47

Merged
VeckoTheGecko merged 7 commits into
mainfrom
python-same
May 7, 2026
Merged

Better ASV Pixi integration#47
VeckoTheGecko merged 7 commits into
mainfrom
python-same

Conversation

@VeckoTheGecko
Copy link
Copy Markdown
Contributor

I think this PR results in better Pixi/ASV integration - where ASV (when providing asv run --python=same) uses the same Pixi installation as in benchmarks pixi environment.

If you want to use a different Parcels installation to run the benchmarks, it should be a simple one line update in pixi.toml from

diff --git a/pixi.toml b/pixi.toml
index 7f964c0..5afba7a 100644
--- a/pixi.toml
+++ b/pixi.toml
@@ -22,7 +22,7 @@ python = ">=3.12"
 intake-xarray = "*"
 py-rattler = "0.15.*"
 asv = ">=0.6.5"
-parcels = { path = "./Parcels" }
+parcels = { path = "/Users/Hodgs004/coding/repos/parcels" } # e.g.
 
 
 [tasks]

This workflow is working well for me. Feel free to give it a spin @erikvansebille and if it works well for you, we can merge into main. This can also wait until you're back from EGU.

@VeckoTheGecko
Copy link
Copy Markdown
Contributor Author

(most of the diff is from the lockfile, which changed significantly because we chance how we install Parcels here)

python=same was not resolving to the right Python environment on the HPC system (instead resolving to a broken python binary path). Something must be broken in the behaviour when setting python=same
This reverts commit 16095a6.

Trying it now, apparently its not a problem? idk anymore
For some reason the platform is acting up with the xgcm python build dep
@erikvansebille
Copy link
Copy Markdown
Member

Yep, this new Pixi workflow seems to work for me! Very nice that I can link to another Parcels pro this way. So do we still need the submodule here?

Comment thread Parcels
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we need the Parcels submodule, or ask users to link to their local Parcels repo by default?

Copy link
Copy Markdown
Contributor Author

@VeckoTheGecko VeckoTheGecko May 7, 2026

Choose a reason for hiding this comment

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

I think having the Parcels submodule is best - but users can definitely override this locally if they want to for debugging. They can even directly modify the submodule and contribute back that way if they want to

@VeckoTheGecko VeckoTheGecko merged commit e7d2e1c into main May 7, 2026
1 check passed
@VeckoTheGecko VeckoTheGecko deleted the python-same branch May 7, 2026 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants