EnergySystemsModellingLab / EnergySystemsModellingLab/MUSE_OS

Change interpolation method for capacity data

Ouverte
#656 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Python
Étoiles
29
Forks
12
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

The `interpolate_capacity` function is currently using linear interpolation. However, as pointed out [here](https://github.com/EnergySystemsModellingLab/MUSE_OS/pull/642#discussion_r1944376005), linear interpolation doesn't make much sense because capacity profiles are step-wise. Forward filling (i.e. taking the previous specified value) would make more sense.

We can simply change `interpolate_capacity` to return `data.reindex(year=year, method="ffill")`

I've given this a try, but some of the results changed and I'd like to understand why before implementing this change

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par la fonction interpolate_capacity et examinez ses appelants afin de comprendre comment les données de capacité sont indexées et pourquoi le remplissage vers l’avant modifie les résultats. Comparez l’interpolation linéaire actuelle avec data.reindex(year=year, method="ffill") pour tous les profils de capacité concernés ; le travail est considéré comme terminé lorsque le comportement de l’interpolation est compris et que le résultat attendu est confirmé avant toute modification.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
data-engineering
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.