Skip to content

fixed best_delta to initialise based on the shape of delta, not the shape of the input samples - #180

Open
andreybarsky wants to merge 1 commit into
pralab:mainfrom
andreybarsky:separating-sample-and-delta-shape
Open

fixed best_delta to initialise based on the shape of delta, not the shape of the input samples#180
andreybarsky wants to merge 1 commit into
pralab:mainfrom
andreybarsky:separating-sample-and-delta-shape

Conversation

@andreybarsky

Copy link
Copy Markdown

Small bugfix but helps for the case where the perturbation being optimised does not have the same dimensions as the original input itself (e.g. when you are optimising a kernel that is convolved with the input to create the adversarial image)

@maurapintor

Copy link
Copy Markdown
Collaborator

Checking with @zangobot if this breaks anything in his domain 🧱

@zangobot

Copy link
Copy Markdown
Contributor

In the other libraries that use secml-torch, we have re-defined some of those. So it seems legit to me.

@maurapintor

Copy link
Copy Markdown
Collaborator

The pipeline failed as setuptools broke some things. @andreybarsky, you should be able to pull from main and merge the fixing changes, and that should fix the CI pipeline execution.

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.

3 participants