-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Heat Map: don't show bow attributes if flag hidden=True #5306
Comments
What do you think @PrimozGodec? |
Hmm, it is a difficult one. On one side it would be nice to be consistent and remove hidden attributes. On the other side, what if someone wants to see embeddings on the heatmap. Then she/he would need to go through each attribute in the edit domain and change Another solution would be to remove hidden attributes from the heatmap and add a checkbox to the embedding widgets same as in the Bag of words widget. I am actually more for this second option. This way we give the user the option to hide some attributes and still he can check embeddings with the heatmap. |
Right, I forgot embeddings are also hidden. Yes, this would make sense - adding a checkbox to Doc Emb would make the most sense. And of course hiding hidden attributes in Heat Map. |
I think this was the proposed solution:
|
Partially fixed in biolab/orange3-text#852. |
Some visualization widgets will ignore bow attributes if they have a hidden=True flag (in attributes of attributes). Heat Map would benefit from the same setup: ignoring bow attributes (and showing just sentiment, topics, etc.).
Add the same option as Box Plot has it for hiding bow attributes.
Keep it as is.
The text was updated successfully, but these errors were encountered: