Skip to content

fix: update CFR logic#4616

Merged
abeizn merged 2 commits into
apache:mainfrom
merico-ai:issue-4610
Mar 8, 2023
Merged

fix: update CFR logic#4616
abeizn merged 2 commits into
apache:mainfrom
merico-ai:issue-4610

Conversation

@Startrekzky
Copy link
Copy Markdown
Contributor

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have read through the Contributing Documentation.
  • I have added relevant tests.
  • I have added relevant documentation.
  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary

Update the logic of change failure rate

Does this close any open issues?

Closes #4610

Screenshots

0583d4a6-45f7-468b-80f1-780b8eb66f22

@Startrekzky Startrekzky requested a review from abeizn March 8, 2023 09:45
Copy link
Copy Markdown
Contributor

@abeizn abeizn left a comment

Choose a reason for hiding this comment

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

LGTM

@abeizn abeizn merged commit 6f71ad9 into apache:main Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][Metric] The way to calculate Change Failure Rate is not accurate

3 participants