Skip to content

Commit

Permalink
Merge pull request #268 from AUS-DOH-Safety-and-Quality/assurance-ico…
Browse files Browse the repository at this point in the history
…n-fixes

Fix assurance icon detection with varying and missing alt. target values
  • Loading branch information
andrjohns authored Apr 1, 2024
2 parents 1750d66 + dc73197 commit 53290b4
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/Outlier Flagging/assuranceIconToDraw.ts
Original file line number Diff line number Diff line change
@@ -1,14 +1,17 @@
import type { viewModelClass } from "../Classes";

export default function assuranceIconToDraw(viewModel: viewModelClass): string {
if (viewModel.inputSettings.settings.spc.chart_type === "run") {
return "none";
}
const imp_direction: string = viewModel.inputSettings.settings.outliers.improvement_direction;
const alt_target: number = viewModel.inputSettings.settings.lines.alt_target;
const N: number = viewModel.controlLimits.ll99.length - 1;
const alt_target: number = viewModel.controlLimits?.alt_targets?.[N];

if (alt_target === null || imp_direction === "neutral") {
return "none";
}

const N: number = viewModel.controlLimits.ll99.length - 1;
const impDirectionIncrease: boolean = imp_direction === "increase";

if (alt_target > viewModel.controlLimits.ul99[N]) {
Expand Down

0 comments on commit 53290b4

Please sign in to comment.