ntnuiotenergy / OpenEMPIRE

Stochastic linear program for investments in the European power system
https://openempire.readthedocs.io/en/latest/
MIT License
16 stars 15 forks source link

Change seasons definition #35

Open martinhjel opened 5 months ago

martinhjel commented 5 months ago

https://github.com/ntnuiotenergy/OpenEMPIRE/blob/8008ee00024ad183cfeea6a5a8536df1e537c9e7/scenario_random.py#L14

This is the current mapping of months to seasons, that follow the astronomical seasons:

if season == "winter":
    return [1, 2, 3]
if season == "spring":
    return [4, 5, 6]
if season == "summer":
    return [7, 8, 9]
if season == "fall":
    return [10, 11, 12]

Wouldn't it make more sense to use the meteorological seasons instead? I.e.

if season == "winter":
    return [1, 2, 12]
if season == "spring":
    return [3, 4, 5]
if season == "summer":
    return [6, 7, 8]
if season == "fall":
      return [9, 10, 11]