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

Experiments HogQL rewrite #24914

Open
19 of 31 tasks
jurajmajerik opened this issue Sep 11, 2024 · 2 comments
Open
19 of 31 tasks

Experiments HogQL rewrite #24914

jurajmajerik opened this issue Sep 11, 2024 · 2 comments
Labels
enhancement New feature or request feature/experimentation Feature Tag: Experimentation

Comments

@jurajmajerik
Copy link
Contributor

jurajmajerik commented Sep 11, 2024

Trends

  • Set up a query runner
  • Retrieve Experiment
  • Prepare Trend query
    • Display type
    • Start/end dates
    • Flag breakdown
  • Prepare exposure query
    • Events/actions
    • Start/end dates
    • Flag breakdown
    • Default feature flag exposure filter
  • Handle math aggregation path
  • Add statistical calculations
  • Custom errors
  • Filter out test accounts
  • Copy over existing tests
  • Port holdout groups

Funnels

  • Set up a query runner
  • Retrieve Experiment
  • Prepare Funnel query
    • Display type
    • Start/end dates
    • Flag breakdown
    • Reset properties
  • Add statistical calculations
  • Custom errors
  • Filter out test accounts
  • Copy over existing tests
  • Port holdout groups

Schema migration

  • Add a metrics field
  • Validate metrics field
  • Migrate Experiment records

Frontend

  • Call /query for primary metric
  • Call /query for secondary metrics
  • Set insight (Form)
  • Set insight (MetricSelector)
  • Set insight (ActionFilter)
  • ^^Repeat for the Trends exposure query
  • ^^Repeat for secondary metrics
  • Set query received from the Insight to filters/metrics
  • Ensure backwards compatibility (transform filters to query)
  • Retrieve from cache vs. recalculate
@abigailbramble
Copy link

https://posthoghelp.zendesk.com/agent/tickets/19665 and https://posthoghelp.zendesk.com/agent/tickets/19431
both seeing experiment results affected by anonymous events.
It looks like experiments are unintentionally filtering out anonymous people: see this thread
This will be fixed by the above migration.

@darkopia
Copy link

darkopia commented Nov 6, 2024

+1 to this. https://posthoghelp.zendesk.com/agent/tickets/20108
In this instance, removing any person property filters lead to matching results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature/experimentation Feature Tag: Experimentation
Projects
None yet
Development

No branches or pull requests

3 participants