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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Enhancement] Introduce express API #46

Open
roma-glushko opened this issue Jan 20, 2021 · 1 comment 路 May be fixed by #74
Open

[Enhancement] Introduce express API #46

roma-glushko opened this issue Jan 20, 2021 · 1 comment 路 May be fixed by #74

Comments

@roma-glushko
Copy link

Thank you for maintaining ideas of keras-vis 馃檹

I'm currently trying to inspect my CNN and it seems like here is the minimal amount of code it's needed to plot feature maps:

Screenshot 2021-01-20 at 21 00 28

It seems to be possible to shortcut. I propose to introduce an express API for solving the most frequent use cases like:

  • plot all/selected feature maps of a convolution layer
  • plot all/selected filters of a convolution layer
  • plot dense layer activation
  • plot a class activation
  • plot silency map

Sklearn and Yellowbrick have great examples of express API:

This would dramatically reduce learning curve and increase adoption of the library.

@keisen
Copy link
Owner

keisen commented Jan 31, 2021

Hi, @roma-glushko . Thank you for sharing your great ideas.

This would dramatically reduce learning curve and increase adoption of the library.

I think so too.
I believe they are useful for tf-keras-vis users including me.

Sklearn and Yellowbrick have great examples of express API:

But I'm NOT familiar with them, so providing the features may take a time.
Could you help us to design and implement them.

@keisen keisen linked a pull request Aug 13, 2021 that will close this issue
6 tasks
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 a pull request may close this issue.

2 participants