REopt_API icon indicating copy to clipboard operation
REopt_API copied to clipboard

Published 20 hours ago • verified publisher icon NREL

Sync /job/models defaults with API V2 instead of V1

Open Bill-Becker opened this issue 3 years ago • 3 comments

At least some of the defaults for Django model fields such as PV.installed_cost_per_kw and PV.om_cost_per_kw are based on v1 defaults, and they should be changed to v2 defaults.

Bill-Becker avatar Sep 27 '22 16:09 Bill-Becker

Found 10 fields where v1 defaults were different from v2/stable defaults. I will check these in job/models as part of pass-messages branch. image

rathod-b avatar Sep 27 '22 21:09 rathod-b

Thanks!

Bill-Becker avatar Sep 27 '22 23:09 Bill-Becker

Updated misaligned defaults in https://github.com/NREL/REopt_API/pull/370/commits/996ce723a5cf1e7e781e1fe68a73fbf2f3761150.

rathod-b avatar Sep 28 '22 14:09 rathod-b