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

🐛 (marimekko) drop entities without data #3455

Closed
wants to merge 1 commit into from

Conversation

sophiamersmann
Copy link
Member

@sophiamersmann sophiamersmann commented Apr 5, 2024

  • Fixes a bug where Marimekko charts offered entities for selection that are not included in the chart
  • Example: this chart offers "West Germany" for selection but we don't have any data for West Germany for any of the years
  • Solution:
    • Marimekko charts should use the transformed table for the entity selector (they're similar to scatter plots and slope charts in that regard, as all entities are on screen for a selected year)
    • There was a small bug in the transformTable function: We also don't show a "No data" area if we don't have a x-dimension

SVG tester

  • After making this change, it becomes apparent that tolerance is applied twice, and some entities are displayed for years when they shouldn't be
  • For example: United Arab Emirates show up on staging but not on live
    • United Arab Emirates has data for a single year (2013), and the indicator's tolerance is set to 5
    • In production, United Arab Emirates shows up for the years 2008-2018 (correct), but on staging shows up for 2003-2022 (10 years to both sides instead of 5)
    • I think this happens because tolerance is applied in the transformTable function and then again when filtering for the currently displayed year (see next PR)

Copy link
Member Author

sophiamersmann commented Apr 5, 2024

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

Join @sophiamersmann and the rest of your teammates on Graphite Graphite

@sophiamersmann sophiamersmann force-pushed the marimekko-missing-data branch 2 times, most recently from 1d955ae to 30b5057 Compare April 5, 2024 21:12
@sophiamersmann sophiamersmann changed the base branch from marimekko-showNoDataArea to master April 5, 2024 21:12
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.

1 participant