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

Hide Non-Significant Conversion Rate #579

Merged
merged 4 commits into from
Jun 24, 2024
Merged

Hide Non-Significant Conversion Rate #579

merged 4 commits into from
Jun 24, 2024

Conversation

trieloff
Copy link
Contributor

The current conversion rate display is a bit noisy. With this change,
no conversion rate will be shown for most facets. If the facet has
a significantly different conversion rate from the baseline, then it
will be shown. The greater the significance, the greater the opacity of
the label. This allows you to quickly spot facets (or segments) that
are worthy of attention.

  • feat(rum-explorer): add significance flag to conversion rates display
  • fix(rum-explorer): better styling for interesting and significant conversion rate differences

@aem-code-sync aem-code-sync bot temporarily deployed to hide-nonsig-convrate June 21, 2024 10:14 Inactive
Copy link

aem-code-sync bot commented Jun 21, 2024

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link
Collaborator

@akalfas akalfas left a comment

Choose a reason for hiding this comment

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

We should consistently look at conversion rates that are against visits.

@@ -216,6 +228,19 @@ export default class ListFacet extends HTMLElement {
const conversionspan = document.createElement('span');
conversionspan.className = 'extra';

// we need to divide the totals by average weight
// so that we don't overestimate the significance
const avgWeight = this.dataChunks.totals.pageViews.weight
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
const avgWeight = this.dataChunks.totals.pageViews.weight
const avgWeight = this.dataChunks.totals.visits.weight

tools/rum/elements/list-facet.js Outdated Show resolved Hide resolved
tools/rum/elements/list-facet.js Outdated Show resolved Hide resolved
Co-authored-by: Andrei Kalfas <akalfas@adobe.com>
@aem-code-sync aem-code-sync bot temporarily deployed to hide-nonsig-convrate June 24, 2024 06:35 Inactive
Co-authored-by: Andrei Kalfas <akalfas@adobe.com>
@trieloff trieloff merged commit 39f6317 into main Jun 24, 2024
3 of 4 checks passed
@trieloff trieloff deleted the hide-nonsig-convrate branch June 24, 2024 06:36
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.

2 participants