Repository navigation
PeriodPartition breaks EMGUI #55
Description
Activity
The problem is within the processing of
EnergyModelsGUIwhich does not fully support the new time profile types and subdivisions. The reason for that is thatpartition_durationcan be different for different type of nodes. As a consequence, it cannot be directly incorporated within theTimePeriodas it is the case for all other potential time structures. Subsequently, aPartitionProfilecannot be indexed by aTimePeriodas the information of the partition is not accessible as this level, despite being theoretically able to use them as indices.That being said, it requires a rework of the plotting functionality to work properly for all nodes that have variables or parameters that require
PeriodPartition. As I do not consider it in this case prudent to wait for the implementation inEMGUI(which may take a bit of work), I say that we instead still merge PR 12 inEnergyModelsFlexand provide a warning in theREADME.mdand the documentation instead.This problem also leads to not being able to load saved data that is indexed over a partition period. To this end, you must exclude the result files for these variables from the folder you load..
PeriodPartition, introduced in TimeStruct 0.9.12, creates a bug when used together with EMGUI. Running the GUI on the introduced tests of this PR illustrates this with the errorIdeally, a solution to this should be first implemented in EMGUI before this PR can be merged.