Allow separate square feet adjusters to be applied independently

[akselx]

It is possible that the run_setup.yaml file contains both run_telecommute_strategy and sqft_per_job_adjusters flags set as True. However, the code is currently set up such that base sqft adjusters (sqft_per_job_adjusters) are currently only applied if the flag run_telecommute_strategy in run_setup.yaml is set to False:

bayarea_urbansim/baus/variables.py

Line 160 in 098d42f

if run_setup["run_telecommute_strategy"] and year != base_year:

Otherwise the following elif statement is not evaluated - ideally these could be multiplicative - with a base adjustment, and then the strategy adjustment - though it depends on whether the telecommute adjustments are all inclusive - bundling in the base adjustments on the data side.

bayarea_urbansim/baus/variables.py

Line 163 in 098d42f

elif run_setup["sqft_per_job_adjusters"]:

[akselx]

@theocharides let me know if you think this is a general issue or just one related to my use case for leveraging sqft_per_job_adjusters for the exogenous adjustments.

[theocharides]

Hi @akselx - I think I need more code comments here to start, since it's not very straightforward. Also when I last looked at this I thought I saw a typo where the code comments for these lines are swapped.

The idea was that either telecommute rates get applied to future years, or the default rate gets applied, but in any case you can see it in the inputs based on what's turned on in run_setup.yaml without doing multiplication. But I see how that means you can't apply both at the same time without embedding the defaults in the future years.