Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to poetry #1080

Open
wants to merge 14 commits into
base: devel
Choose a base branch
from
Open

Switch to poetry #1080

wants to merge 14 commits into from

Conversation

MartinBubel
Copy link
Contributor

The issues regarding dependency restrictions and pep-related updates are stacking so I decided it may be time to continue with this act of modernization.

@MartinBubel MartinBubel self-assigned this Jul 15, 2024
@MartinBubel
Copy link
Contributor Author

Hi @lawrennd ,
I have prepared this PR to migrate to python-poetry, which would allow for a better dependency tracking and communication (considering the increasing number of issues related to numpy and maptlotlib compatibility).

However, it seems like the combination of Apple + M-Series chip still fails (CI tests). Unfortunately, debugging on CI runners is tedious. I've gotten access to an old macbook with intel chip and I cannot reproduce the issue there so it most likely is related to apples new M-series chips.

For me, there are three options and I would like to get your opinion on them:

  1. Drop macos support for now. This is not a nice thing to do for end-users, though I would suspect that academic and industrial users might be able to live with it.
  2. Find some cloud service that provides apple silicon runners, like hello-m1 or wait until a friend borrows me a mac.
  3. Stick with setup.py for now.

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.

1 participant