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

Added Poetry support #20

Merged
merged 2 commits into from
May 10, 2021
Merged

Added Poetry support #20

merged 2 commits into from
May 10, 2021

Conversation

chinmayshah99
Copy link
Member

@chinmayshah99 chinmayshah99 commented May 9, 2021

Description

Added support for poetry. Poetry is a package management tool that helps to setup your environment super quickly. More on it here: https://python-poetry.org/

Checklist

@chinmayshah99 chinmayshah99 added the Type: New Feature ➕ Introduction of a completely new addition to the codebase label May 9, 2021
@chinmayshah99 chinmayshah99 assigned dvadym and unassigned dvadym May 9, 2021
@chinmayshah99 chinmayshah99 requested a review from dvadym May 9, 2021 17:36
Copy link
Collaborator

@dvadym dvadym left a comment

Choose a reason for hiding this comment

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

Thanks! I have a question (mostly to understand more about poetry :) )

pyspark = "^3.1.1"
absl-py = "^0.12.0"
dataclasses = {version = "^0.8", python = "~3.6"}
numpy = "<1.21"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just wondering, why <1.21 version limitation? (which currently cover all current versions). Maybe it's better to specify lower bound (something like more than 1.17 or 1.20).

Copy link
Member Author

Choose a reason for hiding this comment

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

Numpy has dropped support for python 3,6 starting 1.21.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks , it makes sense.

pyspark = "^3.1.1"
absl-py = "^0.12.0"
dataclasses = {version = "^0.8", python = "~3.6"}
numpy = "<1.21"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks , it makes sense.

@dvadym dvadym merged commit bfd04a2 into OpenMined:main May 10, 2021
@chinmayshah99 chinmayshah99 deleted the poetry-support branch May 10, 2021 14:51
jspacek pushed a commit to jspacek/PipelineDP that referenced this pull request May 15, 2021
* updated gitignore
* added poetry
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: New Feature ➕ Introduction of a completely new addition to the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants