Skip to content
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

Add MIFGSM Attack #717

Merged
merged 7 commits into from
Nov 9, 2023
Merged

Add MIFGSM Attack #717

merged 7 commits into from
Nov 9, 2023

Conversation

lcycode
Copy link
Contributor

@lcycode lcycode commented Oct 27, 2023

This code implements the fixed epsilon attack, MI-FGSM (https://openaccess.thecvf.com/content_cvpr_2018/html/Dong_Boosting_Adversarial_Attacks_CVPR_2018_paper.html).
MI-FGSM is a classic method that demonstrates superior and robust performance compared to PGD. It's worth noting that MI-FGSM serves as a baseline for many SOTA research papers. Therefore, implementing MI-FGSM proves to be a convenient choice for researchers. Please feel free to share any suggestions for further improvements to the code.

(I close the last request, because I rebase the code by mistakes.)

@lcycode
Copy link
Contributor Author

lcycode commented Oct 30, 2023

I've incorporated the test code and successfully passed checks such as black, flake8, and mypy. I hop that these could be beneficial to you.

@zimmerrol zimmerrol self-assigned this Nov 4, 2023
@coveralls
Copy link

coveralls commented Nov 4, 2023

Coverage Status

coverage: 98.47% (+0.02%) from 98.455%
when pulling 72f3381 on lcycode:add_mifgsm_attack
into 475cb98 on bethgelab:master.

@codecov-commenter
Copy link

codecov-commenter commented Nov 4, 2023

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.46%. Comparing base (475cb98) to head (72f3381).
Report is 14 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #717      +/-   ##
==========================================
+ Coverage   98.45%   98.46%   +0.01%     
==========================================
  Files          55       56       +1     
  Lines        3495     3529      +34     
==========================================
+ Hits         3441     3475      +34     
  Misses         54       54              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

foolbox/attacks/mi_fgsm.py Outdated Show resolved Hide resolved
Copy link
Member

@zimmerrol zimmerrol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zimmerrol zimmerrol merged commit b3fcc73 into bethgelab:master Nov 9, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants