One-liner
Expand the indicator paths of dimensions x, size, and color (currently only y is supported)
This would prevent us hardcoding complete ETL paths as in:
|
view.indicators.size = Indicator( |
|
catalogPath="grapher/demography/2024-07-15/population/historical#population_historical" |
|
) |
|
view.indicators.color = Indicator(catalogPath="grapher/regions/2023-01-01/regions/regions#owid_region") |
I think that what needs to change is the logic in build_views and the calls to _expand_indicator_path.
|
view = { |
|
"dimensions": {dim_name: indicator[dim_name] for dim_name in self.dimension_names}, |
|
"indicators": { |
|
"y": self._expand_indicator_path( |
|
indicator.short_name |
|
), # TODO: Add support for (i) support "x", "color", "size"; (ii) display settings |
|
}, |
|
} |
ranked as 4 in importance/annoyance (1-5 scale), from work in #5593
One-liner
Expand the indicator paths of dimensions
x,size, andcolor(currently onlyyis supported)This would prevent us hardcoding complete ETL paths as in:
etl/etl/steps/export/multidim/wid/latest/incomes_wid.py
Lines 233 to 236 in c5e25da
I think that what needs to change is the logic in
build_viewsand the calls to_expand_indicator_path.etl/etl/collection/core/expand.py
Lines 308 to 315 in 811f439
ranked as 4 in importance/annoyance (1-5 scale), from work in #5593