Skip to content

Commit

Permalink
Merge pull request #308 from VectorInstitute/dbe/fixing_jinja_vulnera…
Browse files Browse the repository at this point in the history
…bility

Upgrading peotry lock to include jinja update.
  • Loading branch information
emersodb authored Jan 6, 2025
2 parents 97a829e + 93e0ed5 commit b07e043
Show file tree
Hide file tree
Showing 4 changed files with 3,048 additions and 3,013 deletions.
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ repos:
- id: isort

- repo: https://github.com/pre-commit/mirrors-mypy
rev: v1.13.0
rev: v1.14.0
hooks:
- id: mypy
name: mypy
Expand Down
4 changes: 2 additions & 2 deletions fl4health/parameter_exchange/parameter_packer.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,8 @@ def unpack_parameters(self, packed_parameters: NDArrays) -> Tuple[NDArrays, floa
# The last entry in the parameters list is assumed to be a clipping bound (even if we're evaluating)
split_size = len(packed_parameters) - 1
model_parameters = packed_parameters[:split_size]
clipping_bound = float(packed_parameters[split_size:][0])
return model_parameters, clipping_bound
clipping_bound = packed_parameters[split_size:][0]
return model_parameters, clipping_bound.item()


class ParameterPackerAdaptiveConstraint(ParameterPacker[float]):
Expand Down
Loading

0 comments on commit b07e043

Please sign in to comment.