search

ScottishCovidResponse / SCRCIssueTracking

Central issue tracking repository for all repos in the consortium
6 stars 0 forks source link

Review Time serialization #651

Closed johnnonweiler closed 4 years ago

johnnonweiler commented 4 years ago

Consider improving how the Time class is being serialized - perhaps renaming methods to make it obvious that times are being serialized as days, and throwing an exception if anyone attempts to serialize a time which is not a multiple of 24.

It probably makes sense to look at this at the same time as #650 .

github-actions[bot] commented 4 years ago

Heads up @paulbessell @johnnonweiler - the "Covid_Simulation_Model" label was applied to this issue.