3 ms·
FWIW this was updated before release to also scale down the image before blurring it. We cut the size in half, or cap it to 300x300, whichever is smaller. This
by greysonp 6y ago
FWIW this was updated before release to also scale down the image before blurring it. We cut the size in half, or cap it to 300x300, whichever is smaller. This was to ensure that the effectiveness of the blur isn't reduced on higher-resolution images.
https://github.com/signalapp/Signal-Android/blob/master/app/src/main/java/org/thoughtcrime/securesms/scribbles/UriGlideRenderer.java#L238 https://github.com/signalapp/Signal-Android/blob/master/app/...
- pfortuny 6y agoYou should perform a non-invertible blur though... Or even easier, use the same noise image for all faces. EDIT: come to think of it, you can generate random noise using a palette from the color in the blur area (say, take four or five colors and mix them). Applying convolutional blur for anonymizing is very very risky. Because you might end up with something either invertible or nearly so.