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

add setup metadata to help PyPI flesh out content on pypi package page #22085

Closed
wants to merge 3 commits into from

Conversation

soumith
Copy link
Member

@soumith soumith commented Jun 21, 2019

add setup metadata to help PyPI flesh out content on pypi package page.

Apparently this might help flesh out the "Used By" feature according to @driazati

@pytorchbot pytorchbot added the module: build Build system issues label Jun 21, 2019
'Topic :: Software Development :: Libraries :: Python Modules',
],
license='BSD',
keywords='pytorch machine learning',
Copy link
Collaborator

Choose a reason for hiding this comment

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

machine learning should be machine-learning perhaps? It seems that keywords are treated as space-separated list:
https://packaging.python.org/guides/distributing-packages-using-setuptools/#keywords

setup.py Outdated
'Topic :: Software Development :: Libraries',
'Topic :: Software Development :: Libraries :: Python Modules',
],
license='BSD',
Copy link
Collaborator

Choose a reason for hiding this comment

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

How about BSD-3? This clarifies PyTorch is under 3-clause BSD (not 4-clause or 2-clause).

@soumith
Copy link
Member Author

soumith commented Aug 1, 2019

@pytorchbot rebase this please

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@soumith is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@soumith merged this pull request in dded794.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged module: build Build system issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants