-
Notifications
You must be signed in to change notification settings - Fork 0
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
30 add aquajl #31
base: main
Are you sure you want to change the base?
30 add aquajl #31
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #31 +/- ##
==========================================
- Coverage 41.32% 40.83% -0.49%
==========================================
Files 10 10
Lines 242 240 -2
==========================================
- Hits 100 98 -2
Misses 142 142 ☔ View full report in Codecov by Sentry. |
No changes to the actual code base except formatting. The rest is just related to house keeping/best practices, so don't think there's much to review here, but will request anyway and not bypass the rule @rithik83 |
) | ||
Random.seed!(0) | ||
n_features = length(x) | ||
|
||
# Initialization (stripes of +/-ϵ) | ||
init_δ = rand(w, 1, c) .|> x -> x < 0.5 ? -ϵ : ϵ | ||
init_δ = (x -> x < 0.5 ? -ϵ : ϵ).(rand(w, 1, c)) |
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.
Things like this do not change the functionality of the step, right? (not just this but in general towards all the tweaks within the attack algorithms)
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.
no all of this is just automated formatting using JuliaFormatter.format(".")
FormatCheck.yml
to ensure that test actually fails if necessary