sui icon indicating copy to clipboard operation
sui copied to clipboard

feat(packages/sui-react-web-vitals): rollback to log targets with empty values again

Open nucliweb opened this issue 1 year ago • 2 comments

Description

The empty values are a bug of the library, in principle, we eliminate them for not sending them to Open Search, and eliminate noise, but not having the target empties gives us a distortion of the real percentages of the attributions, and that is why we resubmit them.

nucliweb avatar Mar 07 '24 19:03 nucliweb

how's that? TBH I didn't really get why wouldn't we log those empty values...

The empty values are a bug of the library, in principle, we eliminate them for not sending them to Open Search, and eliminate noise, but not having the target empties gives us a distortion of the real percentages of the attributions, and that is why we resubmit them.

I just added the explanation as a description in the the PR 😜

nucliweb avatar Mar 08 '24 07:03 nucliweb

how's that? TBH I didn't really get why wouldn't we log those empty values...

The empty values are a bug of the library, in principle, we eliminate them for not sending them to Open Search, and eliminate noise, but not having the target empties gives us a distortion of the real percentages of the attributions, and that is why we resubmit them.

I just added the explanation as a description in the the PR 😜

that's what I thought in the first place... if we were to remove empty values, there'd be an overrepresentation of the elements that get their attribution and name correctly

jordevo avatar Mar 08 '24 07:03 jordevo