-
Notifications
You must be signed in to change notification settings - Fork 104
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
[FEAT] Add a plot_topics method to GapEncoder
#1124
Labels
enhancement
New feature or request
Comments
Vincent-Maladiere
changed the title
[Feat] Add a plot_topics method to
[FEAT] Add a plot_topics method to Oct 22, 2024
GapEncoder
GapEncoder
I agree this would be useful; see also #909 (comment) |
Yup!
But can we have a generic methods (ie get_report()), which could be used everywhere in skrub?
|
Can you elaborate a bit on this @GaelVaroquaux? Do you see |
Can you elaborate a bit on this @GaelVaroquaux? Do you see get_report also working for TextEncoder and MinHashEncoder?
Maybe. But to me, it's more a question of something homogeneous in the library: we could have this on the Recipe (or whatever, I don't know).
I would love it if people would have a one-stop command to ask an object: give me a useful visualization of what you have in your belly. Something a bit more powerful than the default repr.
|
Very interesting! Could we list somewhere the kind of visualizations we would like to see for some objects? |
Very interesting! Could we list somewhere the kind of visualizations we would like to see for some objects?
yes let's make a list and do a sprint :)
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem Description
One of the
GapEncoder
's main features is its interpretability. The documentation shows how to plot a heatmap representing activations across topics and their content, but getting this representation right is time-consuming.Feature Description
I suggest we bring this method directly into the
GapEncoder
class.Alternative Solutions
Creating a separate function elsewhere, akin to sklearn.tree.plot_tree, but at the cost of being less practical and visible.
Additional Context
No response
The text was updated successfully, but these errors were encountered: