[BUGFIX] Move reservoir serialization to new class (!28) · Merge requests · Indu / ramsis.datamodel

sarsonl requested to merge feature/21-reservoir_serialization into develop Jan 19, 2022

The requirement of data type output from the _ReservoirSchema is different for RAMSIS (stored as a Column(MutableDict.as_mutable(JSONEncodedDict)) so that it is modifiable as json in the gui) and the worker (stored as a ReservoirGeometryEdges object ).

The bug occurring was that the json was unable to be modified in the gui as it was loaded as the wrong type. The simplest way to fix it i think is to separate this loading functionality into a different class.

Closes #21 (closed)

[BUGFIX] Move reservoir serialization to new class

Merge request reports