Skip to content

Commit

Permalink
Merge branch 'main' into feat/docker-image-main
Browse files Browse the repository at this point in the history
  • Loading branch information
Robert-Steiner authored Sep 9, 2024
2 parents 215efae + 00b384b commit 3ded0f3
Show file tree
Hide file tree
Showing 8 changed files with 32 additions and 15 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -146,8 +146,6 @@ jobs:
if: ${{ github.repository == 'adap/flower' && !github.event.pull_request.head.repo.fork && github.actor != 'dependabot[bot]' }}
run: |
python -m pip install https://${{ env.ARTIFACT_BUCKET }}/py/${{ needs.wheel.outputs.dir }}/${{ needs.wheel.outputs.short_sha }}/${{ needs.wheel.outputs.whl_path }}
- name: Install e2e components
run: pip install .
- name: Download dataset
if: ${{ matrix.dataset }}
run: python -c "${{ matrix.dataset }}"
Expand All @@ -172,7 +170,7 @@ jobs:
run: ./../test_superlink.sh bare sqlite
- name: Run driver test with client authentication
if: ${{ matrix.directory == 'e2e-bare-auth' }}
run: ./../test_superlink.sh bare client-auth
run: ./../test_superlink.sh "${{ matrix.directory }}" client-auth
- name: Run reconnection test with SQLite database
if: ${{ matrix.directory == 'e2e-bare' }}
run: ./../test_reconnection.sh sqlite
Expand Down
2 changes: 1 addition & 1 deletion benchmarks/flowertune-llm/evaluation/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ The default template generated by `flwr new` (see the [Project Creation Instruct

| | MBPP | HumanEval | MultiPL-E (JS) | MultiPL-E (C++) | Avg |
|:----------:|:-----:|:---------:|:--------------:|:---------------:|:-----:|
| Pass@1 (%) | 32.60 | 26.83 | 29.81 | 24.22 | 28.37 |
| Pass@1 (%) | 31.60 | 23.78 | 28.57 | 25.47 | 27.36 |


## Make submission on FlowerTune LLM Leaderboard
Expand Down
2 changes: 1 addition & 1 deletion benchmarks/flowertune-llm/evaluation/general-nlp/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ huggingface-cli login
Download data from [FastChat](https://github.com/lm-sys/FastChat):

```shell
git clone --depth=1 https://github.com/lm-sys/FastChat.git && cd FastChat && git checkout d561f87b24de197e25e3ddf7e09af93ced8dfe36 && mv fastchat/llm_judge/data ../data && cd .. && rm -rf FastChat
git clone https://github.com/lm-sys/FastChat.git && cd FastChat && git checkout d561f87b24de197e25e3ddf7e09af93ced8dfe36 && mv fastchat/llm_judge/data ../data && cd .. && rm -rf FastChat
```


Expand Down
28 changes: 25 additions & 3 deletions doc/source/contributor-how-to-build-docker-images.rst
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ Before we can start, we need to meet a few prerequisites in our local developmen
default values, others must be specified when building the image. All available build arguments for
each image are listed in one of the tables below.

Building the base image
Building the Base Image
-----------------------

.. list-table::
Expand Down Expand Up @@ -65,6 +65,10 @@ Building the base image
- The Flower package to be installed.
- No
- ``flwr`` or ``flwr-nightly``
* - ``FLWR_VERSION_REF``
- A `direct reference <https://packaging.python.org/en/latest/specifications/version-specifiers/#direct-references>`_ without the ``@`` specifier. If both ``FLWR_VERSION`` and ``FLWR_VERSION_REF`` are specified, the ``FLWR_VERSION_REF`` has precedence.
- No
- `Direct Reference Examples`_

The following example creates a base Ubuntu/Alpine image with Python ``3.11.0``,
pip :substitution-code:`|pip_version|`, setuptools :substitution-code:`|setuptools_version|`
Expand All @@ -84,8 +88,8 @@ and Flower :substitution-code:`|stable_flwr_version|`:
In this example, we specify our image name as ``flwr_base`` and the tag as ``0.1.0``. Remember that the build arguments as well
as the name and tag can be adapted to your needs. These values serve as examples only.

Building the SuperLink/SuperNode or ServerApp image
---------------------------------------------------
Building a Flower Binary Image
------------------------------

.. list-table::
:widths: 25 45 15 15
Expand Down Expand Up @@ -130,3 +134,21 @@ After creating the image, we can test whether the image is working:
.. code-block:: bash
$ docker run --rm flwr_superlink:0.1.0 --help
Direct Reference Examples
-------------------------

.. code-block:: bash
:substitutions:
# main branch
git+https://github.com/adap/flower.git@main
# commit hash
git+https://github.com/adap/flower.git@1187c707f1894924bfa693d99611cf6f93431835
# tag
git+https://github.com/adap/flower.git@|stable_flwr_version|
# artifact store
https://artifact.flower.ai/py/main/latest/flwr-|stable_flwr_version|-py3-none-any.whl
2 changes: 1 addition & 1 deletion doc/source/tutorial-quickstart-mlx.rst
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ You can also override the parameters defined in
.. code:: shell
# Override some arguments
$ flwr run . --run-config num-server-rounds=5,lr=0.05
$ flwr run . --run-config "num-server-rounds=5 lr=0.05"
What follows is an explanation of each component in the project you just
created: dataset partition, the model, defining the ``ClientApp`` and
Expand Down
2 changes: 1 addition & 1 deletion doc/source/tutorial-quickstart-pytorch.rst
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ You can also override the parameters defined in the
.. code:: shell
# Override some arguments
$ flwr run . --run-config num-server-rounds=5,local-epochs=3
$ flwr run . --run-config "num-server-rounds=5 local-epochs=3"
What follows is an explanation of each component in the project you just
created: dataset partition, the model, defining the ``ClientApp`` and
Expand Down
5 changes: 1 addition & 4 deletions e2e/test_superlink.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
set -e

case "$1" in
e2e-bare-https)
e2e-bare-https | e2e-bare-auth)
./generate.sh
server_arg="--ssl-ca-certfile certificates/ca.crt --ssl-certfile certificates/server.pem --ssl-keyfile certificates/server.key"
client_arg="--root-certificates certificates/ca.crt"
Expand Down Expand Up @@ -37,14 +37,11 @@ case "$2" in
client_auth_2=""
;;
client-auth)
./generate.sh
rest_arg_superlink=""
rest_arg_supernode=""
server_address="127.0.0.1:9092"
server_app_address="127.0.0.1:9091"
db_arg="--database :flwr-in-memory-state:"
server_arg="--ssl-ca-certfile certificates/ca.crt --ssl-certfile certificates/server.pem --ssl-keyfile certificates/server.key"
client_arg="--root-certificates certificates/ca.crt"
server_auth="--auth-list-public-keys keys/client_public_keys.csv --auth-superlink-private-key keys/server_credentials --auth-superlink-public-key keys/server_credentials.pub"
client_auth_1="--auth-supernode-private-key keys/client_credentials_1 --auth-supernode-public-key keys/client_credentials_1.pub"
client_auth_2="--auth-supernode-private-key keys/client_credentials_2 --auth-supernode-public-key keys/client_credentials_2.pub"
Expand Down
2 changes: 1 addition & 1 deletion src/py/flwr/client/client_app.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,11 +41,11 @@ def _alert_erroneous_client_fn() -> None:

def _inspect_maybe_adapt_client_fn_signature(client_fn: ClientFnExt) -> ClientFnExt:
client_fn_args = inspect.signature(client_fn).parameters
first_arg = list(client_fn_args.keys())[0]

if len(client_fn_args) != 1:
_alert_erroneous_client_fn()

first_arg = list(client_fn_args.keys())[0]
first_arg_type = client_fn_args[first_arg].annotation

if first_arg_type is str or first_arg == "cid":
Expand Down

0 comments on commit 3ded0f3

Please sign in to comment.