Skip to content

Conversation

@VeckoTheGecko
Copy link
Contributor

Fixes #2272

Before:

pixi run -e test-latest 'cd tests && python -c "import parcels"'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'parcels'

After:

pixi run -e test-latest 'cd tests && python -c "import parcels"'

This enables #2216 - how should we time that @erikvansebille to reduce disruption? After #2303 is merged?

@erikvansebille
Copy link
Member

Looks good to me. Why should we wait until #2303? Is this change not completely perpendicular to that PR?

@VeckoTheGecko
Copy link
Contributor Author

This change is, but #2216 isn't - I was wondering about that

@erikvansebille
Copy link
Member

Maybe I don't understand what #2216 then is. Is it 'simply' a rename of the parcels directory to src?

In that case, let's indeed do it after #2303. That PR is ready, except for the failing assert on the UX hash, which fails in many more tests (right?)

@VeckoTheGecko
Copy link
Contributor Author

Is it 'simply' a rename of the parcels directory to src?

Yes.

Let's do after #2303

@VeckoTheGecko
Copy link
Contributor Author

For some reason generating the lock file is hanging (40m at least for a solve). Not sure why since this works locally. Going to see if I can create a minimal example and bring it to the prefix team

@VeckoTheGecko VeckoTheGecko changed the title Update pixi.toml to fix install Update pixi.toml to install Parcels from source Oct 7, 2025
@VeckoTheGecko VeckoTheGecko enabled auto-merge (squash) October 7, 2025 10:59
@VeckoTheGecko VeckoTheGecko merged commit 37cbace into Parcels-code:v4-dev Oct 7, 2025
10 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Parcels development Oct 7, 2025
@VeckoTheGecko VeckoTheGecko deleted the fix-pixi-install branch October 7, 2025 11:02
reint-fischer pushed a commit to reint-fischer/parcels that referenced this pull request Oct 15, 2025
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.

Does pixi default env have an editable install of parcels?

2 participants