PAM logic

[cadop]

@m-brandon-haworth some questions for you on the PAM part

Is this correct, that there shouldn't be an else here:

crowds/exts/siborg.simulate.crowd/siborg/simulate/crowd/models/pam.py

Lines 163 to 168 in eb80d3d

	if tc < Parameters.time_horizon:
	if len(t_pairs) < Parameters.max_neighbors:
	t_pairs.append((tc, j))
	elif tc < t_pairs[0][0]:
	t_pairs.pop()
	t_pairs.append((tc, j))

What's the difference between replacing the earlier collisions with the current collision, once max_neighbors is reached?

crowds/exts/siborg.simulate.crowd/siborg/simulate/crowd/models/pam.py

Lines 166 to 168 in eb80d3d

	elif tc < t_pairs[0][0]:
	t_pairs.pop()
	t_pairs.append((tc, j))

The neighbor order isn't based on distance (its random) so why bother replacing and not just skip?