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

--[WIP][BE/Refactor/Bugfix] Remove deprecated distutils from setup and ab_test #2202

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jturner65
Copy link
Contributor

@jturner65 jturner65 commented Sep 13, 2023

Motivation and Context

Distutils is deprecated and will be removed in python 3.12, which is slated for release Oct 2 2023. This PR removes references to Distutils in HabitatSim python code. It does require the 'packaging' library, so this also needs to be added to requirements.txt. (@Skylion007 is there a better option than this to provide version classes?)

NOTE : Our versions of bullet, assimp and pybind11 all also have distutils references, so we may need to update these libraries before we can claim python 3.12 support. (magnum also does, but I'm sure mosra will address when appropriate).

How Has This Been Tested

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have completed my CLA (see CONTRIBUTING)
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Sep 13, 2023
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from adcd3bc to 562580c Compare September 14, 2023 16:50
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 5 times, most recently from 94b0634 to 7399bf4 Compare September 27, 2023 19:12
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 3 times, most recently from 7c7bdb4 to 246500f Compare October 12, 2023 13:21
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from e7c3f98 to 153c984 Compare October 26, 2023 15:28
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 153c984 to d0490ac Compare November 14, 2023 16:18
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from 36750ac to a73637b Compare November 29, 2023 14:25
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 4 times, most recently from 1d3155f to 3139d0c Compare December 12, 2023 18:13
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 3139d0c to 56298a4 Compare December 18, 2023 17:18
Copy link
Contributor

@Skylion007 Skylion007 left a comment

Choose a reason for hiding this comment

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

We should probably also bump the supported versions for our conda builds.

@Skylion007
Copy link
Contributor

Latest version of pybind11 supports python 3.12. Also latest versions of torch will too now.

@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from a21bc23 to f4075ad Compare January 4, 2024 13:52
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from f6767f7 to 46c8648 Compare January 9, 2024 21:16
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 46c8648 to a7727fc Compare January 22, 2024 21:16
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 4d31035 to a4f746a Compare May 13, 2024 18:49
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from 15049ff to 36cb92d Compare June 18, 2024 13:50
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 36cb92d to bcb3992 Compare June 20, 2024 14:33
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from 708372c to 693af4b Compare July 25, 2024 15:27
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from 336955f to f39421a Compare August 13, 2024 14:01
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from f33e56e to 73fae5e Compare August 25, 2024 15:46
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from 6723fa4 to 4d8fed8 Compare September 5, 2024 12:57
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 4d8fed8 to d18eecb Compare September 10, 2024 13:41
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from d18eecb to 46f1c11 Compare September 17, 2024 14:25
@jturner65 jturner65 changed the base branch from main to dev September 17, 2024 14:26
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from b6a472d to 36f2b91 Compare September 18, 2024 17:16
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 36f2b91 to 9843784 Compare October 29, 2024 21:10
Base automatically changed from dev to main November 4, 2024 16:35
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from ae91b20 to d279d43 Compare November 11, 2024 16:04
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch 2 times, most recently from 9ffa254 to f2c33c8 Compare November 13, 2024 19:55
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from f2c33c8 to 97835a0 Compare November 25, 2024 19:40
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 97835a0 to 13231ea Compare December 4, 2024 15:18
@jturner65 jturner65 force-pushed the RemoveDistutilsFromSetup branch from 13231ea to 759dafe Compare December 11, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants