Skip to content

Commit

Permalink
(fix): Warning for missing values now matching correct counts [skip ci]
Browse files Browse the repository at this point in the history
  • Loading branch information
ArneTR committed Dec 16, 2024
1 parent 77a7974 commit 04cc436
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions frontend/js/helpers/phase-stats.js
Original file line number Diff line number Diff line change
Expand Up @@ -201,9 +201,15 @@ const renderCompareChartsForPhase = (phase_stats_object, phase='[RUNTIME]', run_
/* END BLOCK LABELS*/

if (
(phase_stats_object.comparison_identifiers.length == 2 && detail_data['data'][phase_stats_object.comparison_identifiers[0]]?.values?.length != detail_data['data'][phase_stats_object.comparison_identifiers[1]]?.values?.length)
(phase_stats_object.comparison_identifiers.length == 2
&& (
detail_data['data'][phase_stats_object.comparison_identifiers[0]]?.values?.includes(null)
|| detail_data['data'][phase_stats_object.comparison_identifiers[0]]?.values == null
|| detail_data['data'][phase_stats_object.comparison_identifiers[0]].values.includes(null)
)
)
||
(phase_stats_object.comparison_identifiers.length == 1 && detail_data['data'][phase_stats_object.comparison_identifiers[0]]?.values?.length != run_count)
(phase_stats_object.comparison_identifiers.length == 1 && detail_data['data'][phase_stats_object.comparison_identifiers[0]]?.values?.includes(null))
) {
showWarning(phase, `${metric_name} ${detail_name} was missing from at least one comparison.`)
}
Expand Down

0 comments on commit 04cc436

Please sign in to comment.