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

Update Audio Poisoning Notebook #2089

Merged

Conversation

Swanand-Kadhe
Copy link
Collaborator

Description

This PR updates poisoning_attack_backdoor_audio.ipynb notebook to match with the recently updated audio perturbation classes.

Related to Issue #2052 (fixed) and PR #2053 (merged).

Type of change

Please check all relevant options.

  • Improvement (non-breaking)
  • Bug fix (non-breaking)
  • New feature (non-breaking)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Testing

Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

  • Test A
  • Test B

Test Configuration:

  • OS
  • Python version
  • ART version or commit number
  • TensorFlow / Keras / PyTorch / MXNet version

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Signed-off-by: Swanand Ravindra Kadhe <Swanand.Kadhe@ibm.com>
@codecov-commenter
Copy link

codecov-commenter commented Mar 29, 2023

Codecov Report

Merging #2089 (80e5c16) into main (0cc12f4) will increase coverage by 0.01%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2089      +/-   ##
==========================================
+ Coverage   85.60%   85.61%   +0.01%     
==========================================
  Files         297      297              
  Lines       26515    26515              
  Branches     4867     4867              
==========================================
+ Hits        22697    22702       +5     
+ Misses       2578     2574       -4     
+ Partials     1240     1239       -1     

see 1 file with indirect coverage changes

@beat-buesser beat-buesser self-requested a review March 30, 2023 12:12
@beat-buesser beat-buesser self-assigned this Mar 30, 2023
@beat-buesser beat-buesser added the notebooks Notebooks label Mar 30, 2023
Copy link
Collaborator

@beat-buesser beat-buesser left a comment

Choose a reason for hiding this comment

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

Hi @Swanand-Kadhe Thank you very much, looks good to me.

@beat-buesser beat-buesser merged commit c1859ba into Trusted-AI:main Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notebooks Notebooks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants