Skip to content

Add analytics #245

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

Merged
merged 1 commit into from
Jun 20, 2025
Merged

Add analytics #245

merged 1 commit into from
Jun 20, 2025

Conversation

benjeffery
Copy link
Member

From tskit-dev/tskit#960
This creates public analytics at https://tskit.goatcounter.com/

@benjeffery
Copy link
Member Author

@jeromekelleher Not sure if you get notifications about the site repo. Assume you're ok with analytics data being public? It can be behind a secret key instead if we like.

@jeromekelleher
Copy link
Member

Perfectly happy for them to be public.

However, we should discuss the privacy implications etc. Can you summarise the effect of this change please?

@benjeffery
Copy link
Member Author

This change adds a script which sends data on every page view to "GoatCounter". https://www.goatcounter.com/

They describe themselves as "an open source web analytics platform available as a free donation-supported hosted service or self-hosted app. It aims to offer easy to use and meaningful privacy-friendly web analytics as an alternative to Google Analytics or Matomo"

It doesn't use cookies or store any identifying data, so doesn't need a GDPR notice on the webpages. Session tracking is done by a hash hash(User-Agent + IP) which is only stored in RAM for 8 hours. (Session tracking can also be turned off).

@jeromekelleher
Copy link
Member

OK, SGTM. We should get more buy-in on this before merging though

@benjeffery
Copy link
Member Author

Have discussed on slack and in person with a few folks now, merging.

@benjeffery benjeffery merged commit f35b3da into tskit-dev:main Jun 20, 2025
15 checks passed
@benjeffery benjeffery deleted the analytics branch June 20, 2025 15:48
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