Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor kpi calculation #7

Closed
wants to merge 4 commits into from
Closed

Conversation

janniclas
Copy link
Contributor

Extracted processChildScores functionality in a separate function

In strict mode incomplete results are not considered for further calculation, thus they must be handled identically to Error or Empty results.
Especially, the weights planned for the incomplete node must be distributed between the remaining nodes.
…CalculationStrategy in a separate method.

- removed redundant loops
- added documentation
@janniclas janniclas requested a review from AnakinRaW August 14, 2024 12:18
@janniclas janniclas self-assigned this Aug 14, 2024
@janniclas janniclas closed this Aug 14, 2024
@janniclas janniclas deleted the f-refactorKpiCalculation branch August 14, 2024 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant