-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Send nb_classes to check_and_transform_label_format in inference attacks #1713
Send nb_classes to check_and_transform_label_format in inference attacks #1713
Conversation
Signed-off-by: abigailt <abigailt@il.ibm.com>
Signed-off-by: abigailt <abigailt@il.ibm.com>
…ship attacks as well) Signed-off-by: abigailt <abigailt@il.ibm.com>
Signed-off-by: abigailt <abigailt@il.ibm.com>
Signed-off-by: abigailt <abigailt@il.ibm.com>
Signed-off-by: abigailt <abigailt@il.ibm.com>
…ship attacks as well) Signed-off-by: abigailt <abigailt@il.ibm.com>
Signed-off-by: abigailt <abigailt@il.ibm.com>
… dev_1.11.0_attribute_fixes # Conflicts: # art/attacks/inference/attribute_inference/black_box.py # art/attacks/inference/attribute_inference/true_label_baseline.py
Signed-off-by: abigailt <abigailt@il.ibm.com>
Codecov Report
@@ Coverage Diff @@
## dev_1.11.0 #1713 +/- ##
==============================================
- Coverage 88.04% 84.27% -3.77%
==============================================
Files 260 260
Lines 21442 21448 +6
Branches 3809 3809
==============================================
- Hits 18879 18076 -803
- Misses 1612 2450 +838
+ Partials 951 922 -29
|
Signed-off-by: abigailt <abigailt@il.ibm.com>
if self._values is not None: | ||
self._nb_classes = len(self._values) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need these two lines? Or can we document the definition with len(self._values)
in a comment in line 128?
if self._values is not None: | |
self._nb_classes = len(self._values) |
if self._values is not None: | ||
self._nb_classes = len(self._values) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need these two lines? Or can we document the definition with len(self._values)
in a comment in line 138?
if self._values is not None: | ||
self._nb_classes = len(self._values) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need these two lines? Or can we document the definition with len(self._values)
in a comment in line 114?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @abigailgold Thank you very much for your pull request. I think the updates look good. I have a question about 2 lines in each class if they are required. What do you think?
Signed-off-by: abigailt <abigailt@il.ibm.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @abigailgold Thank you very much. The code looks good to me!
Description
Send nb_classes to check_and_transform_label_format in inference attacks.
Fixes #1712
Type of change
Please check all relevant options.
Testing
New test that sends labels as column vector.
Test Configuration:
Checklist