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.

Sign up for GitHub

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

Jump to bottom

#156 Add sum of truncated values as measure #157

Merged
merged 4 commits into from
Nov 4, 2024
Merged

#156 Add sum of truncated values as measure #157

merged 4 commits into from
Nov 4, 2024

Conversation

Zejnilovic
Copy link
Collaborator

Closes #156

Added just 2 new measurement methods. Sum of truncated normal and absolute values.

@Zejnilovic Zejnilovic self-assigned this Oct 11, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 11, 2024 •
edited
Loading

JaCoCo code coverage report - scala:2.12.15

Overall Project 52.77% -0.86% 🍏
Files changed 57.85% ❌

Module Coverage
atum:core Jacoco Report - scala:2.12.15 54.11% -1.02% ❌
Files
Module File Coverage
atum:core Jacoco Report - scala:2.12.15 MeasurementProcessor.scala 94.08% -0.33% 🍏
ControlType.scala 88.52% -20.49% ❌

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 14, 2024 •
edited
Loading

JaCoCo code coverage report - scala:2.11.12

Overall Project 47.62% -0.67% 🍏
Files changed 70.45% ❌

Module Coverage
atum:core Jacoco Report - scala:2.11.12 49.09% -0.78% ❌
Files
Module File Coverage
atum:core Jacoco Report - scala:2.11.12 MeasurementProcessor.scala 83.3% 🍏
ControlType.scala 79.01% -16.05% ❌

yruslan
yruslan approved these changes Oct 14, 2024
View reviewed changes
Copy link
Collaborator

@yruslan yruslan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@Zejnilovic Zejnilovic mentioned this pull request Oct 17, 2024
Closed
@Zejnilovic Zejnilovic merged commit 17148a7 into master Nov 4, 2024
5 checks passed
@Zejnilovic Zejnilovic deleted the feature/156-calc-sum-truc-values branch November 4, 2024 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yruslan yruslan yruslan approved these changes

@benedeki benedeki Awaiting requested review from benedeki

Assignees

@Zejnilovic Zejnilovic

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add support for sum of truncated values
2 participants
@Zejnilovic @yruslan