Make year selection more secure in transport data creation #1225

FabianHofmann · 2024-08-19T11:39:13Z

Extends on ad100c4 but for the future probably a bit more secure.

Also use index.unique instead of index.levels to avoid duplications and avoid some performance issues around lexsort

Closes #1218

Checklist

I tested my contribution locally and it seems to work fine.
Code and workflow changes are sufficiently documented.
Changed dependencies are added to envs/environment.yaml.
Changes in configuration options are added in all of config.default.yaml.
Changes in configuration options are also documented in doc/configtables/*.csv.
A release note doc/release_notes.rst is added.

use index.unique instead of index.levels where needed

for more information, see https://pre-commit.ci

lisazeyen

Hi, looks all good! Just for the Swiss transport data I would suggest to stick with the hard coded numbers.

scripts/build_energy_totals.py

build_transport_data: select swiss years according to data selection

ca47bec

use index.unique instead of index.levels where needed

FabianHofmann requested a review from fneum August 19, 2024 11:39

[pre-commit.ci] auto fixes from pre-commit.com hooks

54a2d24

for more information, see https://pre-commit.ci

lisazeyen reviewed Aug 19, 2024

View reviewed changes

scripts/build_energy_totals.py Show resolved Hide resolved

lisazeyen merged commit 49053f3 into master Aug 20, 2024
6 checks passed