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

🎉 Add data explaining capability to Insighter #3311

Merged
merged 4 commits into from
Sep 18, 2024

Conversation

larsyencken
Copy link
Collaborator

The insighter has one mode at the moment which uses a chart view.

This PR adds a second, where it attempts to capture all the trends in a raw dataset.

@owidbot
Copy link
Contributor

owidbot commented Sep 18, 2024

Quick links (staging server):

Site Admin Wizard

Login: ssh owid@staging-site-insighter-plus-plus

chart-diff: ✅ No charts for review.
data-diff: ✅ No differences found
Legend: +New  ~Modified  -Removed  =Identical  Details
Hint: Run this locally with etl diff REMOTE data/ --include yourdataset --verbose --snippet

Automatically updated datasets matching weekly_wildfires|excess_mortality|covid|fluid|flunet|country_profile|garden/ihme_gbd/2019/gbd_risk are not included

Edited: 2024-09-18 17:29:37 UTC
Execution time: 13.50 seconds

@larsyencken larsyencken merged commit 27438de into master Sep 18, 2024
7 of 8 checks passed
@larsyencken larsyencken deleted the insighter-plus-plus branch September 18, 2024 19:53
@larsyencken
Copy link
Collaborator Author

Merging without review as part of prep for Stockholm.

@larsyencken
Copy link
Collaborator Author

/cc @danyx23

@larsyencken
Copy link
Collaborator Author

We should simplify that code when we have a new CSV API.

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