Skip to content
This repository has been archived by the owner on Dec 24, 2019. It is now read-only.

scava-overview dashboard : "last facts" vs "recommendations" #419

Closed
mhow2 opened this issue Nov 6, 2019 · 6 comments
Closed

scava-overview dashboard : "last facts" vs "recommendations" #419

mhow2 opened this issue Nov 6, 2019 · 6 comments
Assignees

Comments

@mhow2
Copy link

mhow2 commented Nov 6, 2019

Believing the screenshot, scava-overview dashboard filtered on project XWiki and sorted by date, it seems that "last facts" and "recommendations" are about the same ("recommendations" contains an additional project column).

image

Is there an explanation for it ? Isn't the term "recommendation" about the KB service ?

@valeriocos
Copy link
Member

The recommendations table shows facts about the rascal metrics. If you open the visualization you can see the list of metrics:
captura_125.

On the other hand, last facts shows the facts ordered by date.

@mhow2
Copy link
Author

mhow2 commented Nov 6, 2019

The recommendations table shows facts about the rascal metrics

It seems the last facts show those too, that's the point. both viz look about the same

@valeriocos
Copy link
Member

yes, because the "recommendations" are a subset of all facts. What do you suggest @mhow2 ?

@mhow2
Copy link
Author

mhow2 commented Nov 7, 2019

@borisbaldassari :
"recommendations" metrics is a subset of factoids metrics, eg rascal ones.
"last facts" contains all metrics (that includes rascal ones).

It was the result of the work in crossminer/scava-deployment#42 and especially crossminer/scava-deployment#42 (comment)

I'm trying to understand how "COCOMO" can turn into a recommendation (#421) and if we shouldn't rather leave it for the "last facts" table.
I'm also wondering if we shouldn't simply remove all rascal metrics from "last facts" ? So we consolidate the spirit of have two tables for two separate purpose instead of mixing up.

WDYT ?

@borisbaldassari
Copy link
Member

@mhow2 for the sake of visual consistency, it sounds good to have them separated and not showing up twice. m2c. :-)
As for cocomo, you have to be warned that it's been debunked a couple of times and it may show really weird numbers..

@valeriocos
Copy link
Member

Fixed with #426

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants