[WIP] Build POT against oldest-supported-numpy by davidghiurco · Pull Request #347 · PythonOT/POT · GitHub
Skip to content

[WIP] Build POT against oldest-supported-numpy#347

Closed
davidghiurco wants to merge 5 commits into
PythonOT:masterfrom
davidghiurco:issue346/build-against-oldest-supported-numpy
Closed

[WIP] Build POT against oldest-supported-numpy#347
davidghiurco wants to merge 5 commits into
PythonOT:masterfrom
davidghiurco:issue346/build-against-oldest-supported-numpy

Conversation

@davidghiurco

@davidghiurco davidghiurco commented Feb 11, 2022

Copy link
Copy Markdown
Contributor

Types of changes

Build / setup:

  • Compiling against the oldest possible numpy version possible given the platform, while also still allowing numpy>=1.20 to be used
    ** NOTE: I suspect POT would happily support & compile against even older numpy versions, and maintain forward compatibility with newer numpy, but I'm not sure.
  • Removing legacy code workarounds to build-time dependency problems which have been already fixed by the inclusion of a pyproject.toml in [WIP] POT build without installing cython first #293
  • Set python_requires to >=3.7. Python 3.6 support was dropped in POT release 0.8.1. This configuration is also necessary to help oldest-supported-numpy properly determine what the oldest version of numpy it should use during setup.

Motivation and context / Related issue

Address #346

How has this been tested (if it applies)

PR checklist

  • I have read the CONTRIBUTING document.
  • The documentation is up-to-date with the changes I made (check build artifacts).
  • All tests passed, and additional code has been covered with new tests.
  • I have added the PR and Issue fix to the RELEASES.md file.

@davidghiurco

davidghiurco commented Feb 11, 2022

Copy link
Copy Markdown
Contributor Author

Comment thread setup.py Outdated
@davidghiurco

davidghiurco commented Feb 11, 2022

Copy link
Copy Markdown
Contributor Author

If you change numpy>=1.20 -> numpy>=1.16 in install_requires, you should also change python_requires=">=3.7" -> python_requires=">=3.6", because numpy<1.20 is not compatible with Python 3.7+ anyways.

That is if you want to re-add Python 3.6 support.

Comment thread setup.py Outdated
@rflamary

Copy link
Copy Markdown
Collaborator

Hello @davidghiurco and thank you for the PR. I'm not sure why but the github actions for building the wheels are not triggered so I will probably ope a new PR wfrom this one on a branc of POT to ensure that the wheels are building OK before merging

@davidghiurco

davidghiurco commented Feb 15, 2022

Copy link
Copy Markdown
Contributor Author

@rflamary

Copy link
Copy Markdown
Collaborator

well build all wheels is longer because it build on ARM for linux but both should run, i really believe i need to have the pr on a branch of POT to make it work (but i don't see why it does not run in PR only from the github action conditions...)

@rflamary

Copy link
Copy Markdown
Collaborator

@rflamary rflamary closed this Feb 23, 2022
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