Skip to content

Commit 445687b

Browse files
authored
Merge pull request #52 from DataTools4Heart/fix/sync-changes
function fixed
2 parents bfa8201 + 6ab8573 commit 445687b

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

flcore/models/linear_models/server.py

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,17 @@ def evaluate_held_out(
5959
kwargs: Dict[str, fl.common.Scalar],
6060
config: Dict[str, fl.common.Scalar],
6161
) -> Tuple[float, Dict[str, float]]:
62-
62+
"""
63+
Esta función no tiene razón de ser, para funcionar necesita acceso
64+
al dataset, cosa que no tiene por qué tener. Esta es la idea central
65+
del aprendizaje federado. La infraestructura de DT4H NO permitirá
66+
tener un dataset para llevar a cabo esta prueba. Razón por la que
67+
esta función no tiene propósito de existir. Tiene sentido que exista
68+
en un ambiente simulado con un dataset público, pero no en un despliegue
69+
real.
70+
"""
6371
"""Evaluate the current model on the held-out validation set."""
72+
"""
6473
# Load held-out validation data
6574
client_id = config['held_out_center_id']
6675
# client_id = -1 # kaggle hf
@@ -122,9 +131,9 @@ def evaluate_held_out(
122131
#Add 'centralized' to the metrics to identify them
123132
non_weighted_centralized_metrics = {f"non weighted centralized {key}": non_weighted_centralized_metrics[key] for key in non_weighted_centralized_metrics}
124133
metrics.update(non_weighted_centralized_metrics)
125-
126-
127134
return loss, metrics
135+
"""
136+
return 0, {}
128137

129138

130139
def get_server_and_strategy(config):

0 commit comments

Comments
 (0)