urbansim / urbansim_parcels

New version of urbansim_defaults for compatibility with new developer model
0 stars 1 forks source link

Modify feasibility steps to only instantiate pro forma object once #4

Closed pksohn closed 7 years ago

pksohn commented 7 years ago

A quick fix to something I noticed. Previously, the parcel model (that I designed) was instantiating the SqFtProForma class three separate times in the process of running feasibility. It's not a big slowdown, but there's no need for that. I changed it to create a SqFtProForma object, then pass the object itself to the prepare_parcels_for_feasibility and lookup_by_form steps.