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

%-comparison is way off - not considering imported GA data? #2870

Closed
metmarkosaric opened this issue Apr 25, 2023 Discussed in #2867 · 1 comment · Fixed by #2871
Closed

%-comparison is way off - not considering imported GA data? #2870

metmarkosaric opened this issue Apr 25, 2023 Discussed in #2867 · 1 comment · Fixed by #2871
Assignees

Comments

@metmarkosaric
Copy link
Contributor

Discussed in #2867

Originally posted by Swiss-Mac-User April 23, 2023
The comparison vs previous period of my site are way off, e.g. when using „Current month“ view. Is this a bug in correlation with GA data import, where also native Plausible data has been tracked ?

I started tracking with Plausible last month (March) and it recorded „natively“ 550 pageviews. All the other data was imported from Google Analytics to keep the historical data.

It seems to me, that the data comparison vs last period (increase / decrease) does not take the imported data into account…

  • Current month: 2.9k pageviews
  • Last month: 4.1k pageviews
  • Comparison: should be „-34%“
  • Instead it shows as: „+450%“? (That’s the 550 pageviews natively recorded by Plausible in the last period)

IMG_2158
IMG_2159

@vinibrsl vinibrsl self-assigned this Apr 25, 2023
@vinibrsl
Copy link
Contributor

Oops! This was caused by the refactoring we did for comparisons. I'm currently working on a fix :)

vinibrsl added a commit that referenced this issue Apr 26, 2023
This commit fixes a bug where comparisons would not work with imported data. This is because comparisons copies the source query and modifies the dates only, where it should reevaluate if there is imported data for the new comparison date range.

Closes #2870

Co-authored-by: Adam <[email protected]>
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 a pull request may close this issue.

2 participants