Skip to content

Conversation

@soumith
Copy link
Contributor

@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
Contributor 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.

@xuhdev xuhdev deleted the versioninfo branch August 1, 2019 20:02
@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.

6 participants