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

Update TFF to use the new standard format to describe project metadata declaratively. #4234

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Oct 20, 2023

Update TFF to use the new standard format to describe project metadata declaratively.

See https://peps.python.org/pep-0621/

@copybara-service copybara-service bot force-pushed the cl/572583124 branch 5 times, most recently from 5b95310 to 129ff08 Compare October 25, 2023 20:30
@evgri243
Copy link

evgri243 commented Nov 8, 2023

Just curious. Two questions:

  • does it allow dev installations that way?
  • any way to mark builds platform specific, so that they are marked by python build?

python "tensorflow_federated/tools/python_package/setup.py" bdist_wheel \
--plat-name=manylinux_2_31_x86_64
pip install --upgrade build
python -m build --wheel
Copy link

Choose a reason for hiding this comment

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

What platform is set for the package in this case?

@copybara-service copybara-service bot force-pushed the cl/572583124 branch 3 times, most recently from e5bd080 to a87fc6e Compare March 21, 2024 16:25
@copybara-service copybara-service bot changed the title Update TFF to use the new standardized format to describe project metadata declaratively. Update TFF to use the new standard format to describe project metadata declaratively. Mar 27, 2024
@michaelreneer
Copy link
Collaborator

@evgri243

Just curious. Two questions:

  • does it allow dev installations that way?
  • any way to mark builds platform specific, so that they are marked by python build?

What do you mean by dev installations? Do you mean creating a developer environment typically done using venv and installing the requirements.txt? If yes, then no that is still a separate process this is just to replace the setup.py script.

I think yes. Maybe this is what you are looking for https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#dependencies-and-requirements or possibly https://setuptools.pypa.io/en/latest/userguide/dependency_management.html#platform-specific-dependencies

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants