Repository: Giskard-AI/giskard-oss
Issue: Change the tooltip for the Metamophic test: today the description of the tooltip is the same between test wilcoxxon et t-test
Root cause: Documentation Deficiency


@jmsquare @alexcombessie @andreybavt 
Currently, these are the names we're using:
```yaml
Invariance for Classification 
Invariance for Classification (t-test)
Invariance for Classification (Wilcoxon)
Decreasing for Classification 
Decreasing for Classification (t-test)
Decreasing for Classification (Wilcoxon)
Increasing for Classification 
Increasing for Classification (t-test)
Increasing for Classification (Wilcoxon)
Invariance for Regression 
Decreasing for Regression 
Increasing for Regression 
```
How do you suggest we rename/shorten them?

I propose we remove `for Classification` and `for Regression` as they won't co-exist anyways.
But then the issue is, we'll have a duplication of `Invariance` for example, one that is active, one that is blocked.
<img width="386" alt="Screenshot 2022-12-22 at 17 51 15" src="https://user-images.githubusercontent.com/32709181/209185809-209cf84d-cac3-4bb0-a8d6-4a8bd350649d.png">


Hi @rabah-khalek 

Good ideas. I agree that we should "remove for Classification and for Regression as they won't co-exist anyways."
@andreybavt to me, the tests that are not available should not be shown as "blocked"; they should not be visible.
It is a quick fix; could we do it without waiting for AI test v2?

Also, @rabah-khalek, I would suggest that since we added ({method}) details in parenthesis, we also add the parenthesis to the tests "Increasing/Invariance/Decreasing for Classification". Inside the parenthesis, we could say something like "base" / "simple" / "proportion." 
This way, users can understand the difference between the three types of tests for a given family of test.

Lastly, @andreybavt could reduce the large indentation and use a smaller font on test names so that the name of the test is visible? In the current UI, almost all the test names are cutoff outside the screens.

![CleanShot 2022-12-23 at 19 34 52](https://user-images.githubusercontent.com/4739848/209392633-f856352d-2235-4773-82c0-40bb059f4366.png)


Cheers,

Alex

Is everyone happy with this?
<img width="394" alt="Screenshot 2023-01-30 at 10 54 12" src="https://user-images.githubusercontent.com/32709181/215444880-5d287c9a-2af1-4b7f-b77f-23830dcfb7cc.png">

It would be better if we remove all together the regression tests for classification models like you suggested Alex, instead of having:
<img width="391" alt="Screenshot 2023-01-30 at 10 55 31" src="https://user-images.githubusercontent.com/32709181/215445172-d51bef81-cca8-4938-b333-6381436d8d1e.png">



Agreed on both point! thanks

Alex Combessie
Co-founder & CEO @ *Giskard*
*Eliminate risks of AI biases*
Website: giskard.ai ( https://bit.ly/364GAGi )
*⭐️ Star us on G itHub ( https://bit.ly/37aWG1y )*
( https://bit.ly/37aWG1y )

On Mon, Jan 30, 2023 at 10:55:58, Rabah Abdul Khalek < ***@***.*** > wrote:

> 
> 
> 
> Is everyone happy with this?
> Screenshot 2023-01-30 at 10 54 12 (
> https://user-images.githubusercontent.com/32709181/215444880-5d287c9a-2af1-4b7f-b77f-23830dcfb7cc.png
> )
> 
> 
> 
> It would be better if we remove all together the regression tests for
> classification models like you suggested Alex, instead of having:
> Screenshot 2023-01-30 at 10 55 31 (
> https://user-images.githubusercontent.com/32709181/215445172-d51bef81-cca8-4938-b333-6381436d8d1e.png
> )
> 
> 
> 
> —
> Reply to this email directly, view it on GitHub (
> https://github.com/Giskard-AI/giskard/issues/615#issuecomment-1408305957 )
> , or unsubscribe (
> https://github.com/notifications/unsubscribe-auth/ABEFGCEVANGIESPAQFJEEBDWU6F25ANCNFSM6AAAAAATETEZ5U
> ).
> You are receiving this because you were mentioned. Message ID: <Giskard-AI/giskard/issues/615/1408305957
> @ github. com>
> 
> 
>

Ok to summarise:

- [x] making the metamorphic tests names shorter.
- [ ] verify that the renaming does the job in HuggingFace Spaces. (The origin of the issue was that the name was too long to be shown there).
- [ ] remove all together the regression tests for classification model. @andreybavt is that easy to do? Should we do it before AI test v2 since Giskard is now public on HuggingFace spaces?

For HF we have to rebuild single image

For the HF spaces update, since this will be for release 2.0, no problem if you prefer to delay it!

I'll postpone the auto-test generation improvements to v2 too since this mechanism will be changed in v2 with the "scan" feature