🔨 migrate slope charts / TAS-721

[sophiamersmann]

WIP

A slope chart is either:

• migrated to work as a stand-alone chart
• added to the corresponding line chart and deleted

Migration

• Stand-alone charts
  • The migration chooses a random set of entities to be initially selected
    • It's possible that we select a sub-optimal set of entities that don't have data for the default years
  • It also makes sure that entity selection not disabled and the (line) legend is not hidden
• Line+slope charts
  • The migration adds a "slope" tab to existing line charts if the matching is unambiguous

Step by step

• Run the queries below against the live db
• Merge this PR that runs the migration
• Go through referenced slope charts and update the links to point to the slope tab of the corresponding line chart (todo: google sheet)
• Delete slope charts manually (todo: google sheet)
• Go through stand-alone slope charts and check that their selection is ok (todo: google sheet)
• Run sync script for chart configs in R2 (how do I do that?)

Query: slope charts

with line_charts as (select c.id,
                            c.configId,
                            cc.slug,
                            cc.full ->> '$.dimensions[0].variableId' as variableId,
                            cc.full ->> '$.selectedEntityNames'      as selectedEntityNames
                     from charts c
                              join chart_configs cc on c.configId = cc.id
                     where coalesce(cc.full ->> '$.type', 'LineChart') = 'LineChart'
                       and json_length(cc.full, '$.dimensions') = 1
                       and cc.full ->> '$.isPublished' = 'true'),
     slope_charts as (select c.id,
                             c.configId,
                             cc.slug,
                             cc.full ->> '$.dimensions[0].variableId' as variableId,
                             cc.full ->> '$.selectedEntityNames'      as selectedEntityNames,
                             cc.full ->> '$.includedEntities'         as includedEntities,
                             cc.full ->> '$.excludedEntities'         as excludedEntities
                      from charts c
                               join chart_configs cc on c.configId = cc.id
                      where cc.full ->> '$.type' = 'SlopeChart'
                        and cc.full ->> '$.isPublished' = 'true')
select sc.variableId          as variableId,
       sc.id                  as slopeChartId,
       sc.configId            as slopeConfigId,
       sc.slug                as slopeSlug,
       lc.id                  as lineChartId,
       lc.configId            as lineConfigId,
       lc.slug                as lineSlug,
       sc.selectedEntityNames as slopeEntities,
       lc.selectedEntityNames as lineEntities,
       sc.includedEntities,
       sc.excludedEntities
from slope_charts sc
         left join line_charts lc on sc.variableId = lc.variableId;

Query: References

select pg.slug as gdocsSlug, cc.slug as slopeSlug
from charts c
join chart_configs cc on cc.id = c.configId
join posts_gdocs_links pgl on pgl.target = cc.slug
join posts_gdocs pg on pgl.sourceId = pg.id
where pgl.linkType = 'grapher'
and cc.full ->> '$.type' = 'SlopeChart';

To do

• Update queries once the chartTypes PR landed

[sophiamersmann]

• 🔨 migrate slope charts / TAS-721 #4210 👈 (View in Graphite)
• 🎉 make line/slope switching available to authors / TAS-720 #4207 : 1 other dependent PR (#4266 )
• 🎉 (slope) support facetting / TAS-734 #4247
• ✨ (slope) redesign / TAS-722 #4222
• 🎉 (slope) add tolerance / TAS-719 #4205
• 🎉 (slope) feature parity with line charts / TAS-708 #4192
• 🔨 migrate slope chart configs #4275
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[notion-workspace]

Migrate slope charts

[owidbot]

Quick links (staging server):

Site Dev	Site Preview	Admin	Wizard	Docs

Login: ssh owid@staging-site-migrate-slopes-viz

• Site-screenshots: https://github.com/owid/site-screenshots/compare/migrate-slopes-viz
• SVG tester: https://github.com/owid/owid-grapher-svgs/compare/migrate-slopes-viz

SVG tester:

Number of differences (default views): 538 (65fbbc) ❌
Number of differences (all views): 0 ✅

Edited: 2024-11-26 17:57:28 UTC
Execution time: 1.29 seconds