text visualizations seems to have colormap reversed

[cwmeijer]

See any other RISE or some of the LIME tutorials images for the correct colormap convention

[elboyran]

Should be fixed in the LIME text tutorial and RISE text tutorial indeed!

Colormap - 'bwr'.

Red should indicate a positive contribution towards the decision and blue- a negative contribution.

[SarahAlidoost]

The default colormap is chosen as RdBu, see here. It should be changed to bwr and notebooks should be updated. I can fix it.

[elboyran]

[like] Elena Ranguelova reacted to your message:

…

________________________________ From: SarahAlidoost ***@***.***> Sent: Thursday, May 23, 2024 12:16:06 PM To: dianna-ai/dianna ***@***.***> Cc: Elena Ranguelova ***@***.***>; Comment ***@***.***> Subject: Re: [dianna-ai/dianna] text visualizations seems to have colormap reversed (Issue #755) The default colormap is chosen as RdBu, see here<https://github.com/dianna-ai/dianna/blob/b3518eced5be0a02d196dc5b477a0791281b0a90/dianna/visualization/text.py#L9>. It should be changed to bwr and notebooks should be updated. I can fix it. — Reply to this email directly, view it on GitHub<#755 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAYYBWO5KVVFOK4XOWRN42LZDXMYNAVCNFSM6AAAAABGYVI4CSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRWHE3DEMBRGU>. You are receiving this because you commented.Message ID: ***@***.***>

[elboyran]

Also, we are now not using HTML visualization anymore, right?
Then we should edit the 4. Visualization section of the notebook to remove that claim.

[SarahAlidoost]

Fixed the colormap issue (though notebooks can be simplified now, by not using the colormap explicitly) with a few commits: e0d1ee8; 6e843b1 in #769 , but found out dianna breaks if the sentence ends with a dot :-(

Thanks. Let's submit the "dot" problem in another issue. Currently, the visualization wraps the text if there are dots, see here. However, it has not been tested, see here. Can you please submit the "dot" problem in a separate issue? I can try to fix it.

[elboyran]

[like] Elena Ranguelova reacted to your message:

…

________________________________ From: SarahAlidoost ***@***.***> Sent: Monday, May 27, 2024 11:21:34 AM To: dianna-ai/dianna ***@***.***> Cc: Elena Ranguelova ***@***.***>; Assign ***@***.***> Subject: Re: [dianna-ai/dianna] text visualizations seems to have colormap reversed (Issue #755) Fixed the colormap issue (though notebooks can be simplified now, by not using the colormap explicitly) with a few commits: e0d1ee8<e0d1ee8>; 6e843b1<6e843b1> in #769<#769> , but found out dianna breaks if the sentence ends with a dot :-( Thanks. Let's submit the "dot" problem in another issue. Currently, the visualization wraps the text if there are dots, see here<https://github.com/dianna-ai/dianna/blob/b3518eced5be0a02d196dc5b477a0791281b0a90/dianna/visualization/text.py#L65>. However, it has not been tested, see here<https://github.com/dianna-ai/dianna/blob/b3518eced5be0a02d196dc5b477a0791281b0a90/tests/test_visualization.py#L84>. Can you please submit the "dot" problem in a separate issue? I can try to fix it. — Reply to this email directly, view it on GitHub<#755 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAYYBWIDVGBEN3WFRVEWUGTZEMJL5AVCNFSM6AAAAABGYVI4CSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZTGI3DGOJTHA>. You are receiving this because you were assigned.Message ID: ***@***.***>

[elboyran]

Fixed by commits 643b478, c461e7c and 1302a4f.

@loostrum, @SarahAlidoost , @cwmeijer it's very strange that RISE gives always only positive scores even for very negative reviews! See RISE for text tutorial.. Any ideas?