The Intention behind the unused function `hasSpecificSelection`
chengr4 opened this issue · 4 comments
chengr4 commented
Hi,
In graphs
page, I have found out the function hasSpecificSelection
is never used.
Just curious. Do we keep it for future feature need?
/*
* Returns true if a specific subset of charts is selected by benchmark
* name, profile or scenario. In that case, the regular aligned grid of charts
* will not be shown.
*/
function hasSpecificSelection(selector: GraphsSelector): boolean {
return (
selector.benchmark !== null ||
selector.profile !== null ||
selector.scenario !== null
);
}
path: site/frontend/src/pages/graphs/page.vue
Kobzol commented
Hi, this function is used on lines 75 and 148 of the same file.
chengr4 commented
But we are not able to give benchmark
, profile
and scenario
any value on the graph
page, which means hasSpecificSelection
is always false
.
Kobzol commented
chengr4 commented
Got it! thanks for explanation.