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

Preprocessing function argument needed? #126

Open
michaelsok opened this issue Apr 6, 2020 · 0 comments
Open

Preprocessing function argument needed? #126

michaelsok opened this issue Apr 6, 2020 · 0 comments

Comments

@michaelsok
Copy link

michaelsok commented Apr 6, 2020

Hi, and thanks for the work on interpretability!

It may be a dumb question, but shouldn't the methods like OcclusionSensitivity, SmoothGrad and any interpretability method which use modified inputs have a preprocess_function argument?
I mean, technically, the preprocessing may be a step before the model (such as the preprocess function in keras for VGG-16) but change the distribution of the images.
And usually images are normalized, so applying a grey patch on them (for OcclusionSensitivity) does not make sense (for instance, if we have pixels with range between 0 and 1, the grey patch gives images with pixels equal to 127.5).

So I think a preprocess argument may be needed when a model relies on it before training, but I may have missed something.

In all cases, thanks for the hard work!

@michaelsok michaelsok changed the title Preprocessing function argument needed ? Preprocessing function argument needed? Apr 6, 2020
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

No branches or pull requests

1 participant