Skip to content

fixed the build dependencies to include numpy#1208

Closed
James11222 wants to merge 1 commit intoLSSTDESC:masterfrom
James11222:master
Closed

fixed the build dependencies to include numpy#1208
James11222 wants to merge 1 commit intoLSSTDESC:masterfrom
James11222:master

Conversation

@James11222
Copy link

@James11222 James11222 commented Nov 5, 2024

By including numpy as a build dependency, this fixes build installation issues through the pip and uv.

This fixes isolated installs with pip.

@henryiii helped me figure out this fix!

This should fix #1180

By including numpy as a build dependency, this fixes build installation issues through the pip and uv.
@damonge
Copy link
Collaborator

damonge commented Nov 13, 2024

@James11222 you'll need to merge the latest version of master for the tests to pass. Unless you want to close this and I'll open a PR with your modification.

@damonge damonge mentioned this pull request Nov 14, 2024
@henryiii
Copy link

I'd recommend checking the somewhat poorly named "Always suggest updating pull request branches" in the repo settings. That will add a little button that can be used to merge or rebase with a single click.

@damonge
Copy link
Collaborator

damonge commented Nov 15, 2024

Thanks @henryiii ! I was not aware of this.
I have now merged #1212 , which includes these modifications and giving you both credit for it. I'll close this PR.
Thanks a lot for this!

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.

FindNumpy Error when Installing with Poetry

3 participants