This message was deleted.

That makes sense. Supporting #3 is fairly easy (just need to change some

MIN

to

MAX

in the query). #4 might be trickier since we'll need to remove the group by and add a later aggregation step.

Hi again! 😊 Just realized from this that the end date of the metric is actually not the end date of the experiment as I thought. So our current work-around doesn’t actually work (unless we never update the queries after experiment ended, but we often need to… ). So if we could get the #3 (see👆🏻 ) up and working soon, that would be a huge help! That way first experiment viewed event is the start, and the last experiment viewed + window_length is the end of the metric count. Right? Or do you know how others solve this? Is it only us that want to count conversions in, and only in, the whole time frame?

We have a PR that we're hoping to launch this week - https://github.com/growthbook/growthbook/pull/479

Nice! Thanks for the info and efforts! 😊

Thanks for working on this, Jeremy!

@full-island-88199 @stocky-energy-64916 After thinking this through some more, moving our default behavior from #1 to #4 (multiple conversion windows) might make the most sense. So every exposure event for a user will have its own conversion window. Any metric that happens within any of the windows will be included. Question for you: Are there any situations where you would want to keep the existing behavior? (first exposure event only)

Hello! Yes, I also think that makes the most sense. I can’t think of any situations where we’d need to keep the exiting behavior, but I’ll quickly run it by the rest of our team too 🙂 Thanks for looking into this!

Hi - also actively following this PR/issue. We would also have no need to keep the first option.

Just merged this! I kept the default behavior the same for now, but you can switch it on a per-experiment basis. I'm using the term "*Attribution Models*" to describe the different ways to include/exclude metric conversions in an experiment. • First Exposure - Single conversion window based on the first exposure event (default, current behavior) • All Exposures - Multiple conversion windows, one for each exposure event You can change the attribution model in the Analysis Settings section on experiment pages. Let me know how its working. I want to make this the new default behavior in a future release.