Skip to content

Commit

Permalink
Return cumulative multiplier, not last multiplier
Browse files Browse the repository at this point in the history
supply_and_demand was returning the multipliers only from the last
iteration of the price updates. This fixes it to accumulate the
multipliers and return that.
  • Loading branch information
jiffyclub committed Oct 6, 2014
1 parent 9bee525 commit cb9abe6
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 6 deletions.
13 changes: 9 additions & 4 deletions urbansim/models/supplydemand.py
Original file line number Diff line number Diff line change
Expand Up @@ -124,16 +124,21 @@ def supply_and_demand(
bm = base_multiplier.loc[alt_segmenter]
bm.index = alt_segmenter.index
alternatives[price_col] = alternatives[price_col] * bm
base_multiplier = base_multiplier.copy()

for _ in range(iterations):
alts_muliplier, submarkets_multiplier = _calculate_adjustment(
lcm, choosers, alternatives, alt_segmenter,
clip_change_low, clip_change_high)
alternatives[price_col] = alternatives[price_col] * alts_muliplier

# if we started with a base ratio, return a cumulative ratio
if base_multiplier is not None:
submarkets_multiplier *= base_multiplier
# might need to initialize this for holding cumulative multiplier
if base_multiplier is None:
base_multiplier = pd.Series(
np.ones(len(submarkets_multiplier)),
index=submarkets_multiplier.index)

base_multiplier *= submarkets_multiplier

logger.debug('finish: calculating supply and demand price adjustment')
return alternatives[price_col], submarkets_multiplier
return alternatives[price_col], base_multiplier
4 changes: 2 additions & 2 deletions urbansim/models/tests/test_supplydemand.py
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ def test_supply_and_demand(
index=filtered_alts.index) ** 5)
pdt.assert_series_equal(
submarkets_multiplier,
pd.Series([w, x, y, z], index=['w', 'x', 'y', 'z']))
pd.Series([w, x, y, z], index=['w', 'x', 'y', 'z']) ** 5)


def test_supply_and_demand_base_ratio(
Expand All @@ -144,4 +144,4 @@ def test_supply_and_demand_base_ratio(
index=filtered_alts.index) ** 6)
pdt.assert_series_equal(
submarkets_multiplier,
pd.Series([w, x, y, z], index=['w', 'x', 'y', 'z']) ** 2)
pd.Series([w, x, y, z], index=['w', 'x', 'y', 'z']) ** 6)

0 comments on commit cb9abe6

Please sign in to comment.