Set results UI denominator correctly when it is 0 #2448
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If a denominator is 0 for a ratio metric, we should return that rather than fall back to users. This PR does that.
Because the stats engine will always return denominator = 0 for non-ratio metrics, we have to use the kind of logic I've built into the FE here.
This PR also allows for any non-binomial metric to be a ratio denominator (for classic metrics), not just count metrics.
Before (non-sensical results):
![Screenshot 2024-04-29 at 12 20 20 PM](https://private-user-images.githubusercontent.com/5298599/326561495-a0c470c2-844c-4f81-baad-72304f7675fa.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1ODQ5NTYsIm5iZiI6MTcyMDU4NDY1NiwicGF0aCI6Ii81Mjk4NTk5LzMyNjU2MTQ5NS1hMGM0NzBjMi04NDRjLTRmODEtYmFhZC03MjMwNGY3Njc1ZmEucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcxMCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MTBUMDQxMDU2WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9OTU1NGJiYWIyODE4ZjU1MWU0NjQyMTdlNTA5ZGI5ZjIwZmQ5NjZiY2EyYmFhNGFiOTU5MmFlZGU4MGRmMDM4MSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.RxOAD_LWErvoCx350rBn7VpCkBWvx1745XvOlJWulJ8)
After (more sensical results):
![Screenshot 2024-04-29 at 12 19 59 PM](https://private-user-images.githubusercontent.com/5298599/326561554-00957efb-a3d4-4643-b8ff-93388b8c237f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1ODQ5NTYsIm5iZiI6MTcyMDU4NDY1NiwicGF0aCI6Ii81Mjk4NTk5LzMyNjU2MTU1NC0wMDk1N2VmYi1hM2Q0LTQ2NDMtYjhmZi05MzM4OGI4YzIzN2YucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcxMCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MTBUMDQxMDU2WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MmUxYTk4ODk5MzU3N2IwODU5MTRmYTUxN2M5MzY1YmU4NTk3YWMzY2JkNzRjMDc2NmFiNWRiZmM3YjdhMDU2NCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.fY9fTeXaQq5N9w_5UNoawpenEHKXvWRRVPzYhKDg1Po)