From 042a33a008f18d600d2dfe8d146c761e29c805f6 Mon Sep 17 00:00:00 2001 From: Lucas Lange Date: Thu, 10 Jun 2021 11:09:53 +0200 Subject: [PATCH] fix max_auc for summary without slices Before: shows the AUC of the result with the max attacker advantage Expected and fixed: shows the AUC of the result with max AUC --- .../membership_inference_attack/data_structures.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tensorflow_privacy/privacy/privacy_tests/membership_inference_attack/data_structures.py b/tensorflow_privacy/privacy/privacy_tests/membership_inference_attack/data_structures.py index e8f9f5f..6482d73 100644 --- a/tensorflow_privacy/privacy/privacy_tests/membership_inference_attack/data_structures.py +++ b/tensorflow_privacy/privacy/privacy_tests/membership_inference_attack/data_structures.py @@ -678,7 +678,7 @@ class AttackResults: summary = [] # Summary over all slices - max_auc_result_all = self.get_result_with_max_attacker_advantage() + max_auc_result_all = self.results.get_result_with_max_auc() summary.append('Best-performing attacks over all slices') summary.append( ' %s (with %d training and %d test examples) achieved an AUC of %.2f on slice %s'