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

Support for Python 3.9 on 1.0 branch #1219

Merged

Conversation

shagunsodhani
Copy link
Contributor

Motivation

Support for Python 3.9 for 1.0 branch.

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

CI

Related Issues and PRs

(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

#1062

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 18, 2020
@@ -18,6 +18,7 @@
"License :: OSI Approved :: MIT License",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
# "Programming Language :: Python :: 3.9",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this commented out?
If there is a task, remove this line.

@@ -19,6 +19,7 @@
"Programming Language :: Python :: 3.6",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
# "Programming Language :: Python :: 3.9",
Copy link
Collaborator

Choose a reason for hiding this comment

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

same.

@@ -17,6 +17,7 @@
"License :: OSI Approved :: MIT License",
"Programming Language :: Python :: 3.7",
"Programming Language :: Python :: 3.8",
# "Programming Language :: Python :: 3.9",
Copy link
Collaborator

Choose a reason for hiding this comment

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

same.

shagunsodhani and others added 2 commits December 18, 2020 10:56
* Fix importlib error when using py3.9

* Fix lint errors

* Add a method to match lines based on regex

* fix lint error

* bump hydra version in discovery test plugin

* Add python 3.9 in setup.py

* Skip python 3.9 tests on windows

* use assert_text_same when lengths of actual and expected output do not match

* Remove dependency on hydra version

* Check python version to decide if importlib_resources should be imported

* Update the install script for plugins to reflect support for python 3.9

* pin virtualenv to 20.0.33 for windows

* Fix lint error

* Enable CI for 3.9 on windows

* Update circle ci config

* Update circle ci config

* Fix issues with windows

* Skip Ray test for Python 3.9

* Incorporate feedback

* Fix circleci config

* Fix lint errors

* Fix circleci config

* Fix some ci errors

* Fix lint error

* Fix lint error

* Update setup.py for some plugins

* Disable windows 9 support

* Ax plugin doesnt support 3.9 for now

* Nevergrad python 3.9 support depends on Scikit-learn

* Add news entries

* Accidentally removed support for optuna on windows

* Incorporate feedback

* Disable windows test for optuna
@shagunsodhani shagunsodhani merged commit c2f0fe2 into facebookresearch:1.0_branch Dec 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants