8000 added script for plausible analytics by emmanuelle · Pull Request #4893 · scikit-image/scikit-image · GitHub
[go: up one dir, main page]
Skip to content

added script for plausible analytics #4893

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Aug 10, 2020
Merged

added script for plausible analytics #4893

merged 4 commits into from
Aug 10, 2020

Conversation

emmanuelle
Copy link
Member

At the moment we have a script corresponding to Matomo Analytics, where we have a 21-day trial plan. However, Juan recently discovered Plausible Analytics, which is an open source alternative to Matomo. Both solutions are open-source and don't sell your data. Plausible seems to be cheaper.

This PR adds the script for plausible. The idea is to compare both solutions and to take a paid plan before the end of the trial plan.

@jni
Copy link
Member
jni commented Aug 7, 2020

Why did everything break?

@emmanuelle
Copy link
Member Author

Why did everything break?

Excellent question :-). I'm investigating.

@emmanuelle
Copy link
Member Author

@jni I think it was random CircleCI failures... it seems to be ok now.

@jni
Copy link
Member
jni commented Aug 7, 2020

They are random... And somewhat concerning:

https://travis-ci.org/github/scikit-image/scikit-image/jobs/715749296#L4329-L4359

anyway do you want to uncomment the lines again? =)

@emmanuelle
Copy link
Member Author

They are random... And somewhat concerning:

https://travis-ci.org/github/scikit-image/scikit-image/jobs/715749296#L4329-L4359

anyway do you want to uncomment the lines again? =)

In fact I'll remove them for good if you don't mind, because it seems they are not necessary for visualizing the artifact (I copied these lines from another project).

@emmanuelle
Copy link
Member Author

Anything else I need to do here? I'd like to merge this one soon so that we have some overlap between the free plans of matomo and plausible, in order to compare.

grlee77
grlee77 approved these changes Aug 9, 2020
View reviewed changes
Copy link
Contributor
@grlee77 grlee77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's try it

@sciunto sciunto mentioned this pull request Aug 10, 2020
Closed
@jni jni merged commit 904b145 into scikit-image:master Aug 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@grlee77 grlee77 grlee77 approved these changes

Assignees

@jni jni

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@emmanuelle @jni @grlee77
0