🎉 (slope) support facetting / TAS-734

[sophiamersmann]

Adds facetting support for slope charts.

If there are many facets it could be nice to declutter them, but I don't have time for that now.

[sophiamersmann]

• 🎉 make line/slope switching available to authors / TAS-720 #4207 : 2 dependent PRs (#4210 , #4266 )
• 🎉 (slope) support facetting / TAS-734 #4247 👈 (View in Graphite)
• ✨ (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]

Implement facetting for slope charts

[owidbot]

Quick links (staging server):

Site Dev	Site Preview	Admin	Wizard	Docs

Login: ssh owid@staging-site-slope-facets-viz

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

SVG tester:

Number of differences (default views): 1364 (3ccd51) ❌
Number of differences (all views): 486 (2ffe19) ❌

Edited: 2024-12-11 12:34:31 UTC
Execution time: 1.37 seconds

[marcelgerber]

There's a weird thing going on where on http://staging-site-slope-charts/grapher/antibiotic-usage-by-surveillance-category?tab=slope&time=2020..2022, the line connectors are only shown when Uganda is removed - even though the chart size itself is the same no matter what:

[marcelgerber]

Looks good, except for the bug I just commented about 🙌

No worries if you can't address that bug - it's not a dealbreaker!

[sophiamersmann]

This happened because slope charts without data used a default domain extent of [-1, 1]. I pushed a fix!

[sophiamersmann]

Merge activity

• Dec 11, 7:47 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Dec 11, 7:58 AM EST: Graphite rebased this pull request as part of a merge.
• Dec 11, 7:59 AM EST: A user merged this pull request with Graphite.