Skip to content

Commit b51c67b

Browse files
committed
Add LUS property back to ConFIG
- With the new way we generate matrices, this test seems to pass.
1 parent ef99df6 commit b51c67b

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

docs/source/docs/aggregation/index.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ In TorchJD, an aggregator is a class that inherits from the abstract class
4040
- |yes|
4141
* - :doc:`ConFIG <config>`
4242
- |no|
43-
- |no|
43+
- |yes|
4444
- |yes|
4545
* - :doc:`Constant <constant>`
4646
- |no|

tests/unit/aggregation/test_config.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@
33

44
from torchjd.aggregation import ConFIG
55

6-
from ._property_testers import ExpectedStructureProperty
6+
from ._property_testers import ExpectedStructureProperty, LinearUnderScalingProperty
77

88

99
# For some reason, some permutation-invariance property tests fail with the pinv-based
1010
# implementation.
1111
@mark.parametrize("aggregator", [ConFIG()])
12-
class TestConfig(ExpectedStructureProperty):
12+
class TestConfig(ExpectedStructureProperty, LinearUnderScalingProperty):
1313
pass
1414

1515

0 commit comments

Comments
 (0)