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

[SL-1802] [Bug] join_to_timespine not applied for conversion metrics #1048

Closed
2 tasks done
courtneyholcomb opened this issue Feb 26, 2024 · 0 comments · Fixed by #1047
Closed
2 tasks done

[SL-1802] [Bug] join_to_timespine not applied for conversion metrics #1048

courtneyholcomb opened this issue Feb 26, 2024 · 0 comments · Fixed by #1047
Assignees
Labels
bug Something isn't working triage Tasks that need to be triaged

Comments

@courtneyholcomb
Copy link
Contributor

courtneyholcomb commented Feb 26, 2024

Is this a new bug in metricflow?

  • I believe this is a new bug in metricflow
  • I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior

If a conversion metric specifies that either of it's input measures should join to time spine, that join never occurs.

Expected Behavior

The join to time spine should occur, ensuring that there is a row for each date in the query.

Steps To Reproduce

Add join_to_timespine: true to either measure in a conversion metric's YAML. Query the metric, and you'll see that the output SQL never joins to time spine. Example YAML:

metric:
  name: visit_buy_conversion_rate_7days_fill_nulls_with_0
  description: conversion rate on visits-buys on a 7 day window, filling nulls with 0 for both input measures
  type: conversion
  type_params:
    conversion_type_params:
      base_measure:
        name: visits
        fill_nulls_with: 0
        join_to_timespine: true
      conversion_measure:
        name: buys
        fill_nulls_with: 0
        join_to_timespine: true
      window: 7 days
      entity: user
      calculation: conversion_rate

Relevant log output

No response

Environment

- OS:
- Python:
- dbt:
- metricflow:

Which database are you using?

No response

Additional Context

No response

SL-1802

@courtneyholcomb courtneyholcomb added bug Something isn't working triage Tasks that need to be triaged labels Feb 26, 2024
@courtneyholcomb courtneyholcomb self-assigned this Feb 26, 2024
@courtneyholcomb courtneyholcomb changed the title [Bug] join_to_timespine not applied for conversion metrics [SL-1802] [Bug] join_to_timespine not applied for conversion metrics Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage Tasks that need to be triaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant