-
Notifications
You must be signed in to change notification settings - Fork 7
Closed
Labels
code qualityCode readability or structure is improvedCode readability or structure is improveddocumentationImprovements or additions to documentationImprovements or additions to documentation
Description
For most time series, the time column is sufficient for uniquely addressing rows. Furthermore the combination of time and coordinate is a unique key for weather data.
Thus, we do not need (and do not want, due to its size) the redundant column uuid.
Documentation also needs to be updated accordingly, of course.
Affected input data:
TimeSeriesEntry- class can probably be removed
- affects weather data, coordinate data and "individual time series" (power, power & heat etc.)
ResultEntity- can no longer extend
UniqueEntity - affects system participant results, asset result, flex options result, thermal entity results
- can no longer extend
Examples
Apparent power time series would have the columns:
p,q,time
instead of
uuid,p,q,time
Cosmo weather before:
uuid,coordinateid,time,diffuseirradiance,directirradiance,temperature,winddirection,windvelocity
after:
coordinateid,time,diffuseirradiance,directirradiance,temperature,winddirection,windvelocity
Related discussion
cc @t-ober @danielfeismann fyi
t-ober and danielfeismann
Metadata
Metadata
Assignees
Labels
code qualityCode readability or structure is improvedCode readability or structure is improveddocumentationImprovements or additions to documentationImprovements or additions to documentation