Skip to content

Conversation

Connor1996
Copy link
Collaborator

@Connor1996 Connor1996 commented Oct 13, 2025

I encountered a build error when running pdm run build-ext, then I finally found that it's due to old mlx version in venv which is not consistent with the version in pdm.lock.

Should perform pdm sync after updating pdm.lock, so adding a pre_run hook to run it automatically before running any command.

@Connor1996 Connor1996 changed the title build: perform pdm sync before running build: perform pdm sync before running commands Oct 13, 2025
@Connor1996 Connor1996 requested a review from skyzh October 13, 2025 05:28
Copy link
Owner

@skyzh skyzh left a comment

Choose a reason for hiding this comment

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

I wonder if we should ask students to run that before each test in the writeup. It feels like silently sync will make things confusingly slow..?

@Connor1996
Copy link
Collaborator Author

pdm sync is quite fast when there is nothing to update

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